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

Replace dummy HTTPConsumer with a working one #3347

Merged
merged 1 commit into from Apr 30, 2023
Merged

Replace dummy HTTPConsumer with a working one #3347

merged 1 commit into from Apr 30, 2023

Conversation

LudvigHz
Copy link
Member

@LudvigHz LudvigHz commented Apr 28, 2023

It just responds with 405 method not allowed. IMO this is better than constant warnings when something tries to do an http request at it.

Fixes ABA-457

It just responds with 405 method not allowed. IMO this is better than
constant warnings when something tries to do an http request at it.
@codecov
Copy link

codecov bot commented Apr 28, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (1bed2ee) 88.34% compared to head (3e3eb39) 88.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3347   +/-   ##
=======================================
  Coverage   88.34%   88.34%           
=======================================
  Files         661      661           
  Lines       20951    20951           
=======================================
  Hits        18509    18509           
  Misses       2442     2442           
Impacted Files Coverage Δ
lego/apps/websockets/routing.py 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@LudvigHz LudvigHz added backend bug-fix Pull requests that fix a bug review-needed Pull requests that need review small-fix Pull requests that fix something small labels Apr 28, 2023
@linear
Copy link

linear bot commented Apr 28, 2023

ABA-457 Fix incorrect daphne http configuration

https://abakus.sentry.io/issues/4133264456/?referrer=slack

This is not breaking but just very annoying. I also suspect that all of these messages are coming from either traefik or docker swarm doing health c hecks. These checks are also then incorrectly setup and should be fixed.

@ivarnakken ivarnakken added approved Pull requests that have been approved and removed review-needed Pull requests that need review labels Apr 29, 2023
@LudvigHz LudvigHz merged commit b4e7679 into master Apr 30, 2023
2 of 3 checks passed
@LudvigHz LudvigHz deleted the http-daphne branch April 30, 2023 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved backend bug-fix Pull requests that fix a bug small-fix Pull requests that fix something small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants