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

Without Spring Security #1

Merged
merged 1 commit into from
Aug 9, 2024
Merged

Conversation

jzheaux
Copy link
Contributor

@jzheaux jzheaux commented Aug 8, 2024

This PR removes Spring Security and demonstrates that the empty header error happens without it.

I noticed that sometimes a different error comes up, also equally inscrutable, related to flawed header writing.

Other than the reported empty headers error, I sometimes saw this:

Caused by: org.springframework.web.reactive.function.UnsupportedMediaTypeException: Content type 'application/octet-stream' not supported for bodyType=com.github.david0.streamedheaderproblem.FooController$Identifier

@david0
Copy link
Owner

david0 commented Aug 9, 2024

Yes, that reproduces the problem too.

I noticed even the payload is screwed up sometimes:

POST /foo HTTP/1.1
accept-encoding: gzip
user-agent: ReactorNetty/1.1.20
host: localhost:8080
transfer-encoding: chunked
Accept: application/x-ndjson

HTTP/1.1 200 
Vary: Origin
Vary: Access-Control-Request-Method
Vary: Access-Control-Request-Headers
X-Content-Type-Options: nosniff
X-XSS-Protection: 0
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Pragma: no-cache
Expires: 0
Content-Type: application/x-ndjson
X-Frame-Options: DENY
Date: Fri, 09 Aug 2024 05:49:24 GMT

7
{"i":0}
1


0

@david0 david0 merged commit a2b5637 into david0:main Aug 9, 2024
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.

2 participants