Skip to content

[WebAuthn] Replace deprecated origin= with allowed_origins=#13252

Merged
garyhtou merged 3 commits intomainfrom
copilot/web-authn-handle-deprecation-message
Apr 18, 2026
Merged

[WebAuthn] Replace deprecated origin= with allowed_origins=#13252
garyhtou merged 3 commits intomainfrom
copilot/web-authn-handle-deprecation-message

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 23, 2026

Summary of the problem

The webauthn gem emits a deprecation warning at boot: WebAuthn.origin= is deprecated and will be removed in future. Please use WebAuthn.allowed_origins= instead.

Describe your changes

  • config/initializers/webauthn.rb: Replace all config.origin = assignments with config.allowed_origins = (array form) across staging, production, and development/test environments.
  • spec/support/web_authn_support.rb: Update WebAuthn.configuration.originWebAuthn.configuration.allowed_origins.first to match the new API.
# Before
config.origin = "https://#{Credentials.fetch(:LIVE_URL_HOST)}"

# After
config.allowed_origins = ["https://#{Credentials.fetch(:LIVE_URL_HOST)}"]
Original prompt

This section details on the original issue you should resolve

<issue_title>[WebAuthn] Handle deprecation message</issue_title>
<issue_description>```
DEPRECATION WARNING: WebAuthn.origin= is deprecated and will be removed in future. Please use `WebAuthn.allowed_origins=` instead that also allows configuring multiple origins per Relying Party


## Comments on the Issue (you are @copilot in this section)

<comments>
</comments>


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: YodaLightsabr <76178582+YodaLightsabr@users.noreply.github.com>
Agent-Logs-Url: https://github.com/hackclub/hcb/sessions/e98c2252-b2ba-4215-811c-66d8a9e6101b
Copilot AI changed the title [WIP] Fix deprecation warning for WebAuthn origin [WebAuthn] Replace deprecated origin= with allowed_origins= Mar 23, 2026
Copilot AI requested a review from YodaLightsabr March 23, 2026 00:12
@YodaLightsabr YodaLightsabr marked this pull request as ready for review March 30, 2026 14:10
@YodaLightsabr YodaLightsabr requested a review from a team March 30, 2026 14:10
@YodaLightsabr YodaLightsabr requested a review from garyhtou as a code owner March 30, 2026 14:10
@garyhtou garyhtou added this pull request to the merge queue Apr 18, 2026
Merged via the queue into main with commit 8df69e5 Apr 18, 2026
17 checks passed
@garyhtou garyhtou deleted the copilot/web-authn-handle-deprecation-message branch April 18, 2026 07:16
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.

[WebAuthn] Handle deprecation message

3 participants