Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

minor fixes (flask_wtf.CSRFProtect imports, csrf.init_app for unknown symbols)#704

Merged
andrea-guarino-sonarsource merged 1 commit intomasterSonarSource/sonar-python:masterfrom
fixup/SONARPY-668SonarSource/sonar-python:fixup/SONARPY-668Copy head branch name to clipboard
May 6, 2020
Merged

minor fixes (flask_wtf.CSRFProtect imports, csrf.init_app for unknown symbols)#704
andrea-guarino-sonarsource merged 1 commit intomasterSonarSource/sonar-python:masterfrom
fixup/SONARPY-668SonarSource/sonar-python:fixup/SONARPY-668Copy head branch name to clipboard

Conversation

@andrey-tyukin-sonarsource
Copy link
Contributor

  • add flask_wtf.CSRFProtect to FQNs that deactivate the issue message
  • loosen the requirements on csrf.init_app invocations - allow purely lexical matches
  • further standardize issue messages

@sonarsource-next
Copy link

Kudos, SonarQube Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andrea-guarino-sonarsource andrea-guarino-sonarsource merged commit 08670b1 into master May 6, 2020
@andrea-guarino-sonarsource andrea-guarino-sonarsource deleted the fixup/SONARPY-668 branch May 6, 2020 12:09
hashicorp-vault-sonar-prod bot pushed a commit that referenced this pull request Dec 10, 2025
…rivialTypeInference run (#704)

GitOrigin-RevId: b6338a652bd65cfddf91769b4dc875cec6c501c8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.