Skip to content

Update MLH OAuth endpoints and bump gem patch version to 4.1.1 - #18

Merged
erinosher merged 5 commits into
mainfrom
copilot/update-oauth-endpoints
May 20, 2026
Merged

Update MLH OAuth endpoints and bump gem patch version to 4.1.1#18
erinosher merged 5 commits into
mainfrom
copilot/update-oauth-endpoints

Conversation

Copilot AI commented May 20, 2026

Copy link
Copy Markdown
Contributor

MLH OAuth authorization and token exchange moved to new hosts, so the strategy defaults needed to be updated to avoid routing auth flows to deprecated endpoints. This PR also applies the corresponding semver patch release bump.

  • OAuth endpoint configuration

    • Updated OmniAuth::Strategies::MLH client options to use:
      • https://www.mlh.com/oauth/authorize
      • https://api.mlh.com/v4/oauth/token
    • Aligned site with the new authorization host.
  • Versioning

    • Bumped gem version from 4.1.0 to 4.1.1 (patch).
    • Updated the version assertion spec to match the new version constant.
  • Regression guard

    • Added/updated strategy spec coverage to assert the configured authorize/token endpoint URLs.
option :client_options, {
  site: 'https://www.mlh.com',
  authorize_url: 'https://www.mlh.com/oauth/authorize',
  token_url: 'https://api.mlh.com/v4/oauth/token',
  auth_scheme: :request_body
}

Copilot AI and others added 3 commits May 20, 2026 15:57
Agent-Logs-Url: https://github.com/MLH/omniauth-mlh/sessions/69ee5d26-8e3b-4225-a5d8-ac2d78e336bf

Co-authored-by: erinosher <4386583+erinosher@users.noreply.github.com>
Agent-Logs-Url: https://github.com/MLH/omniauth-mlh/sessions/fad989cc-26a4-430a-96cc-72ac8d662ecc

Co-authored-by: erinosher <4386583+erinosher@users.noreply.github.com>
@erinosher
erinosher marked this pull request as ready for review May 20, 2026 16:12
Copilot AI review requested due to automatic review settings May 20, 2026 16:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the OmniAuth MLH OAuth2 strategy to use MLH’s new authorization/token endpoints, and bumps the gem patch version accordingly so consumers get the corrected defaults.

Changes:

  • Updated OmniAuth::Strategies::MLH client_options to point to the new authorize_url and token_url hosts.
  • Bumped gem version from 4.1.0 to 4.1.1 and updated the version assertion spec.
  • Added a strategy spec asserting the configured OAuth endpoint URLs.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lib/omniauth/strategies/mlh.rb Updates default OAuth client endpoint configuration to new MLH hosts.
spec/omni_auth/strategies/mlh_spec.rb Adds regression coverage for configured authorize/token URLs.
lib/omniauth-mlh/version.rb Bumps gem version constant to 4.1.1.
spec/omni_auth/mlh_spec.rb Updates version assertion to 4.1.1.
.gitignore Ignores vendor/bundle (common Bundler path).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/omniauth/strategies/mlh.rb Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Agent-Logs-Url: https://github.com/MLH/omniauth-mlh/sessions/b4e68677-023f-4ac5-8d88-b93480d6b899

Co-authored-by: erinosher <4386583+erinosher@users.noreply.github.com>
Copilot AI requested a review from erinosher May 20, 2026 16:41
@erinosher
erinosher merged commit ec66546 into main May 20, 2026
1 check passed
@erinosher
erinosher deleted the copilot/update-oauth-endpoints branch May 20, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants