WSH032 / fastapi-proxy-lib

HTTP/WebSocket proxy for starlette/FastAPI

https://wsh032.github.io/fastapi-proxy-lib/

Apache License 2.0

25 stars 4 forks source link

docs: add example of `Modify (redefine) response only to particular endpoint` #37

Open pavelsr opened 3 months ago

pavelsr commented 3 months ago

Summary

First off all, Sean @WSH032, thanks for this library

I added one more example to Advanced usage

My example can be useful for API A/B testing or hiding particular methods of upstream

Close: #36

Checklist

[x] I've read CONTRIBUTING.md.
[x] I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
[x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
[x] I've updated the documentation accordingly.

WSH032 commented 3 months ago

Hi @pavelsr ! Well done! 🎉 Every pull request targeting the documentation is appreciated! Thank you for your contribution!

I have left some reviews, could you take a look at them?

The CI pipeline encountered some errors, but don't worry, I'll fix it.

WSH032 commented 3 months ago

The CI pipeline encountered some issues from the upstream (codecov/feedback#358).

28 Upgrading to `codecov-action` v4 might resolve this issue. I'll test it out and see if the upgrade works.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.74%. Comparing base (977d9c1) to head (c8533aa). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #37 +/- ## ======================================= Coverage 96.74% 96.74% ======================================= Files 9 9 Lines 461 461 Branches 67 67 ======================================= Hits 446 446 Misses 9 9 Partials 6 6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.