Fix NoMethodError when access_token is nil in callback_phase

[Copilot]

Fixes omniauth#160 - NoMethodError: undefined method 'expired?' for nil:NilClass when OAuth2 authentication fails and build_access_token returns nil.

Changes

• Nil access_token handling: Guard expired? call and fail gracefully with :invalid_credentials instead of crashing
• OAuth2 error namespacing: Fix OAuth2::TimeoutError and OAuth2::ConnectionError constant lookup with :: prefix

Example

Before:

self.access_token = build_access_token
self.access_token = access_token.refresh! if access_token.expired?  # crashes if nil
super

After:

self.access_token = build_access_token
if access_token
  self.access_token = access_token.refresh! if access_token.expired?
  super
else
  fail!(:invalid_credentials, CallbackError.new(:invalid_credentials, "Failed to build access token"))
end

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• coveralls.io
  • Triggering command: /usr/bin/ruby3.2 ruby3.2 /home/REDACTED/work/omniauth-oauth2/omniauth-oauth2/vendor/bundle/ruby/3.2.0/bin/rspec (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

can you fix this issue

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.