Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[providers-fab/v1-5] Invalidate user session on password reset (#45139) #45164

Open
wants to merge 1 commit into
base: providers-fab/v1-5
Choose a base branch
from

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Dec 22, 2024

  • session expire on pass change

  • fix statis checks

  • add tests (cherry picked from commit cf401c4)


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

Hmm. Still something wrong :(

@potiuk potiuk force-pushed the backport-fab-provider-commit branch 2 times, most recently from 82ba954 to c8a3de1 Compare December 22, 2024 21:28
@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

All right ! Now all workflows seems to be triggering !

@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

cc: @shubhamraj-git

Copy link
Member

@gopidesupavan gopidesupavan left a comment

Choose a reason for hiding this comment

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

Do we need any change log update?

@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

Do we need any change log update?

It will be generated automatically - following the usual release provider process.

@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

Ah... I need to backport the click changes too :)

@potiuk potiuk force-pushed the backport-fab-provider-commit branch from c8a3de1 to 72b8e8b Compare December 22, 2024 22:08
…e#45139)

* session expire on pass change

* fix statis checks

* add tests
(cherry picked from commit cf401c4)

Co-authored-by: Shubham Raj <[email protected]>

Avoid 1.1.8 version of msgraph-core (apache#45044)

The 1.1.8 version of msgraph-core is buggy - importing some basic
classes causes import error "ABCMeta" is not subscriptable.

We are removing the version from azure provider dependencies hoping
that it will be fixed in the next version.

microsoftgraph/msgraph-sdk-python-core#781
(cherry picked from commit 3310b86)
@potiuk
Copy link
Member Author

potiuk commented Dec 22, 2024

And some more :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants