Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: resolve test failures caused by new Sinatra behavior #144

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

robbkidd
Copy link
Member

@robbkidd robbkidd commented Dec 2, 2024

Which problem is this PR solving?

Short description of the changes

  • Update the Honeycomb test server stub to allow anybody to talk to it.

Sinatra 4.1.0 introduced a host authorization option. Update
the Honeycomb test server stub to allow any body to talk to it.

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
@robbkidd robbkidd added no-changelog Omit this PR from changelog/release notes. version: no-bump labels Dec 2, 2024
@robbkidd robbkidd requested a review from a team as a code owner December 2, 2024 22:03
@TylerHelmuth TylerHelmuth merged commit cc826fb into main Dec 2, 2024
6 checks passed
@TylerHelmuth TylerHelmuth deleted the tybb.fix-sinatra-use-in-test branch December 2, 2024 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Omit this PR from changelog/release notes. version: no-bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants