Skip to content

Commit

Permalink
Streamline auth setup and clean up deprecated code
Browse files Browse the repository at this point in the history
Removed outdated server and settings files from the specs directory, consolidating configuration into the new `support/settings` path. This reduces redundancy and aligns the setup with best practices in handling authentication configuration. Additionally, unnecessary fetch for authorization code was eliminated from the handler, optimizing parameter parsing. These changes improve maintainability and clarity of the codebase.
  • Loading branch information
eliasjpr committed Oct 10, 2024
1 parent 8efdfa0 commit cd9cbe5
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion spec/spec_helper.cr
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ require "digest"
require "base64"
require "faker"
require "../src/authly"
require "./settings"
require "./support/settings"

process = nil
Spec.before_suite do
Expand Down
File renamed without changes.
File renamed without changes.
1 change: 0 additions & 1 deletion src/authly/handler.cr
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ module Authly
client_id = params.fetch("client_id", "")
client_secret = params.fetch("client_secret", "")
redirect_uri = params.fetch("redirect_uri", "")
authorization_code = params.fetch("code", "")
username = params.fetch("username", "")
password = params.fetch("password", "")
refresh_token = params.fetch("refresh_token", "")
Expand Down

0 comments on commit cd9cbe5

Please sign in to comment.