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

keep listener after erroring with invalid upstream #117

Closed
wants to merge 1 commit into from

Conversation

pires
Copy link
Owner

@pires pires commented Oct 8, 2024

Refs #110

@pires pires self-assigned this Oct 8, 2024
@pires
Copy link
Owner Author

pires commented Oct 8, 2024

@peteski22 i couldn't merge your PR due to merge conflicts so I had to fix it which unfortunately dropped your signature. Would you be so kind as to reset your PR commit history, pull this commit and resign it?

@coveralls
Copy link

coveralls commented Oct 8, 2024

Coverage Status

coverage: 94.136% (+0.03%) from 94.111%
when pulling 54be595 on feat/keep_listening_when_upstream_not_trusted
into cd8a402 on main.

@pires pires force-pushed the feat/keep_listening_when_upstream_not_trusted branch from 562e4b9 to 10741be Compare October 8, 2024 11:38
@pires pires force-pushed the feat/keep_listening_when_upstream_not_trusted branch from 10741be to b323cec Compare October 8, 2024 11:49
@pires pires force-pushed the feat/keep_listening_when_upstream_not_trusted branch from b323cec to 54be595 Compare October 8, 2024 12:08
@peteski22
Copy link
Contributor

Can this PR be closed now and the original #110 be merged (post rebase etc. etc)?

@pires pires closed this Oct 8, 2024
@pires pires deleted the feat/keep_listening_when_upstream_not_trusted branch October 8, 2024 17:56
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