-
Notifications
You must be signed in to change notification settings - Fork 789
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
Deprecate Jsonp Middleware #7285
Deprecate Jsonp Middleware #7285
Conversation
c944f91
to
ceb2b0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for chasing up on this! Looks great, let's point out that we are completely removing it.
server/shared/src/main/scala/org/http4s/server/middleware/Jsonp.scala
Outdated
Show resolved
Hide resolved
Good point. I have accepted your change. |
OK. I will run scalafmt tomorrow ;) |
Co-authored-by: Arman Bilge <armanbilge@gmail.com>
1f6d95d
to
ea83a92
Compare
Pipelines are appeased |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🫡
fixes #6959