Skip to content

[doc] improve clarity for camo proxy parameters #21386

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

Merged
merged 6 commits into from
Oct 11, 2022
Merged

[doc] improve clarity for camo proxy parameters #21386

merged 6 commits into from
Oct 11, 2022

Conversation

bwenrich
Copy link
Contributor

@bwenrich bwenrich commented Oct 9, 2022

Updates the Config Cheat Sheet for the ALLWAYS option in Camo proxy.

  • Clarifies the behavior for true (both HTTP and HTTPS is proxied) vs false (only HTTP is proxied)
  • Minor grammar and typo improvements

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 10, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Oct 10, 2022
@a1012112796 a1012112796 added the type/docs This PR mainly updates/creates documentation label Oct 10, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@b59b0ca). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #21386   +/-   ##
=======================================
  Coverage        ?   47.44%           
=======================================
  Files           ?     1020           
  Lines           ?   139007           
  Branches        ?        0           
=======================================
  Hits            ?    65950           
  Misses          ?    65063           
  Partials        ?     7994           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@wxiaoguang wxiaoguang merged commit ea3c6c6 into go-gitea:main Oct 11, 2022
@bwenrich bwenrich deleted the camo-doc branch October 11, 2022 16:37
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants