Skip to content
This repository has been archived by the owner on Apr 30, 2024. It is now read-only.

Move imports inside of csrf_check #189

Merged
merged 2 commits into from
Jul 6, 2022

Conversation

patkujawa-wf
Copy link
Contributor

Doing this import at the top-level is causing some consumers' tests to have a bad time.

Doing this import at the top-level is causing some consumers' tests to have a bad time.
@patkujawa-wf
Copy link
Contributor Author

Shoot, guess furious/_pkg_meta.py needs bumping in this pr too

@aviary-wf
Copy link

Security Insights

(2) Vulnerable direct dependencies were detected
  • 3 vulns in jinja2 < 2.11.3 via doc/requirements.txt
  • 3 vulns in pygments < 2.7.4 via doc/requirements.txt
  • Action Items


    Questions or Comments? Reach out on Slack: #support-infosec.

    Copy link
    Contributor

    @matthewbelisle-wf matthewbelisle-wf left a comment

    Choose a reason for hiding this comment

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

    Code +1 and QA +1 tested locally and got Successfully inserted Async job.

    @matthewbelisle-wf
    Copy link
    Contributor

    @Workiva/release-management-p

    @colefeisthamel-wf
    Copy link

    RM +1

    Copy link

    @rmconsole-wf rmconsole-wf left a comment

    Choose a reason for hiding this comment

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

    +1 from RM

    @rmconsole4-wk rmconsole4-wk merged commit af4d2a5 into master Jul 6, 2022
    @patkujawa-wf patkujawa-wf deleted the Move-imports-inside-of-csrf_check branch July 6, 2022 14:49
    Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
    Labels
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    None yet

    6 participants