Skip to content

Conversation

@unknwon
Copy link
Member

@unknwon unknwon commented May 31, 2022

Describe the pull request

Link to the issue: fixes #6901

Checklist

  • I agree to follow the Code of Conduct by submitting this pull request.
  • I have read and acknowledge the Contributing guide.
  • I have added test cases to cover the new code.

Followup

  • Upload locale_en-US.ini to Crowdin

@unknwon unknwon requested a review from a team May 31, 2022 06:40
@unknwon unknwon changed the title webhook: revalidate blocked local hostname before each delivery webhook: revalidate local hostname before each delivery May 31, 2022
@codecov
Copy link

codecov bot commented May 31, 2022

Codecov Report

Merging #6988 (1b628f7) into main (90bc752) will decrease coverage by 0.01%.
The diff coverage is 20.00%.

@@           Coverage Diff            @@
##            main   #6988      +/-   ##
========================================
- Coverage   8.90%   8.89%   -0.02%     
========================================
  Files         99      99              
  Lines      13267   13271       +4     
========================================
- Hits        1182    1181       -1     
- Misses     11919   11924       +5     
  Partials     166     166              

@unknwon unknwon merged commit 7885f45 into main May 31, 2022
@unknwon unknwon deleted the jc/IsLocalHostname branch May 31, 2022 07:17
unknwon added a commit that referenced this pull request May 31, 2022
unknwon added a commit that referenced this pull request May 31, 2022
unknwon added a commit that referenced this pull request May 31, 2022
dna2github pushed a commit to dna2fork/gogs that referenced this pull request May 1, 2023
dna2github pushed a commit to dna2fork/gogs that referenced this pull request May 1, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SSRF in webhook

3 participants