zio / zio-http

A next-generation Scala framework for building scalable, correct, and efficient HTTP clients and servers
https://zio.dev/zio-http
Apache License 2.0
787 stars 396 forks source link

Remove internal usage of warning header #2955

Closed 987Nabil closed 1 month ago

987Nabil commented 3 months ago

The warning header is currently used by zio-http. But is deprecated. I think errors should not add any info about stack traces or error messages in headers at all. We should have error messages as opt-in in the body. This should be a setting that can be set globally by a config, as well as by the user for a specific call to Response.error and similar methods.

jdegoes commented 2 months ago

/bounty $300 to remove warning headers and to have opt-in error messages in body.

algora-pbc[bot] commented 2 months ago

💎 $300 bounty • ZIO

Steps to solve:

  1. Start working: Comment /attempt #2955 with your implementation plan
  2. Submit work: Create a pull request including /claim #2955 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to zio/zio-http!

Add a bounty • Share on socials

Attempt Started (GMT+0) Solution
🟢 @987Nabil #3050
algora-pbc[bot] commented 1 month ago

💡 @987Nabil submitted a pull request that claims the bounty. You can visit your bounty board to reward.

algora-pbc[bot] commented 1 month ago

🎉🎈 @987Nabil has been awarded $300! 🎈🎊