Skip to content

Potential fix for 1 code quality finding - #15

Merged
matteius merged 1 commit into
mainfrom
ai-findings-autofix/django_forms_workflows-sso_backends.py
Aug 3, 2026
Merged

Potential fix for 1 code quality finding#15
matteius merged 1 commit into
mainfrom
ai-findings-autofix/django_forms_workflows-sso_backends.py

Conversation

@matteius

@matteius matteius commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This PR applies 1/4 suggestions from code quality AI findings. 3 suggestions were skipped to avoid creating conflicts.

…pilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 3, 2026 20:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the get_saml_config() docstring to document an intentional SAML compatibility default: security.wantAttributeStatement is set to False to support IdPs (e.g., common Google SAML configurations) that may not include an AttributeStatement.

Changes:

  • Added a docstring “Compatibility note” explaining why wantAttributeStatement defaults to False.
  • Documented that projects can override the default security configuration via FORMS_WORKFLOWS_SAML["security"].

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@matteius
matteius marked this pull request as ready for review August 3, 2026 20:50
@matteius
matteius merged commit 9299f1c into main Aug 3, 2026
12 checks passed
@matteius
matteius deleted the ai-findings-autofix/django_forms_workflows-sso_backends.py branch August 3, 2026 20:50
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