Skip to content

Conversation

@ssunday
Copy link

@ssunday ssunday commented Oct 23, 2025

Stakeholder Overview (learn more)

Allow Rails API apps to use this gem. Also updates testing/CI flow.

Risk Estimate (learn more)

  • ⚠️ Low confidence

Changes

  • Checks if helper method defined before attempting to use it
  • Also adds new config option to prevent loading the location headers module
  • Rest is CI/Test related.
Updated Dependencies
  • Updated appraisal, rails endpoint
Library-Specific
  • Increment the changelog (CHANGELOG.md)
  • Increment the version number (lib/version.rb)
  • Release & Tag the version above in Github

@ssunday ssunday force-pushed the location-headers-api-handling branch from 6f80987 to e5af761 Compare October 23, 2025 15:52
include ActionPack::Cloudfront::LocationHeaders
end

test '#request_eu? - includes when has helper method' do
Copy link
Author

Choose a reason for hiding this comment

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

not sure of better way to test

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.

1 participant