fix: Don't include csrf_token in GET-only form #6549

pselkirk · 2023-10-26T21:30:56Z

codecov · 2023-10-26T22:08:46Z

Codecov Report

Merging #6549 (42ddc7b) into main (f6a2d8c) will increase coverage by 0.15%.
Report is 43 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6549      +/-   ##
==========================================
+ Coverage   88.70%   88.86%   +0.15%     
==========================================
  Files         290      284       -6     
  Lines       40440    40253     -187     
==========================================
- Hits        35874    35772     -102     
+ Misses       4566     4481      -85

Files	Coverage Δ
ietf/doc/views_conflict_review.py	`97.05% <100.00%> (-0.07%)`	⬇️

... and 16 files with indirect coverage changes

jennifer-richards

lgtm

fix: Don't include csrf_token in GET-only form

42ddc7b

rjsparks requested a review from jennifer-richards November 6, 2023 16:47

jennifer-richards approved these changes Nov 6, 2023

View reviewed changes

rjsparks merged commit 2bec81d into ietf-tools:main Nov 7, 2023
9 checks passed

github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: Don't include csrf_token in GET-only form #6549

fix: Don't include csrf_token in GET-only form #6549

pselkirk commented Oct 26, 2023

codecov bot commented Oct 26, 2023

jennifer-richards left a comment

fix: Don't include csrf_token in GET-only form #6549

fix: Don't include csrf_token in GET-only form #6549

Conversation

pselkirk commented Oct 26, 2023

codecov bot commented Oct 26, 2023

Codecov Report

jennifer-richards left a comment

Choose a reason for hiding this comment