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

Remove unencrypted sap client credentials #2217

Merged

Conversation

MueezKhan246
Copy link
Contributor

@MueezKhan246 MueezKhan246 commented Aug 27, 2024

…Khan/Added-Encrypted-Credentials-Column-In-SAP-Configuration
…Khan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration
…Khan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration
@MueezKhan246 MueezKhan246 changed the base branch from master to MueezKhan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration August 27, 2024 12:56
@MueezKhan246 MueezKhan246 force-pushed the MueezKhan/Remove-Unencrypted-SAP-Client-Credentials branch from 25a323d to 32bd53a Compare August 27, 2024 12:58
…mn-In-SAP-Configuration' of github.com:openedx/edx-enterprise into MueezKhan/Remove-Unencrypted-SAP-Client-Credentials
…guration-From-Code' of github.com:openedx/edx-enterprise into MueezKhan/Migration-For-Removing-Decrypted-Client-Secret-In-SAP-Configuration
…Khan/Added-Encrypted-Client-Secret-In-SAP-Configuration
…Khan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration
…ation' of github.com:openedx/edx-enterprise into MueezKhan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration
…Khan/Data-Migration-For-Encrypted-Credentials-Column-In-SAP-Configuration
…Khan/Removing-Unencrypted-Credentials-From-Model
…mn-In-SAP-Configuration' of github.com:openedx/edx-enterprise into MueezKhan/Removing-Unencrypted-Credentials-From-Model
…Khan/Removing-Unencrypted-Credentials-From-Model
…of github.com:openedx/edx-enterprise into MueezKhan/Remove-Unencrypted-SAP-Client-Credentials
…lable' of github.com:openedx/edx-enterprise into MueezKhan/Removing-Unencrypted-Credentials-From-Model
…of github.com:openedx/edx-enterprise into MueezKhan/Remove-Unencrypted-SAP-Client-Credentials
…Khan/Making-Unencrypted-SAP-Client-Credentials-Nullable
…lable' of github.com:openedx/edx-enterprise into MueezKhan/Removing-Unencrypted-Credentials-From-Model
…of github.com:openedx/edx-enterprise into MueezKhan/Remove-Unencrypted-SAP-Client-Credentials
Copy link
Member

@hamzawaleed01 hamzawaleed01 left a comment

Choose a reason for hiding this comment

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

Thumb of trust 👍 I'm sure you've double checked that it's not being used somewhere else now.

Base automatically changed from MueezKhan/Removing-Unencrypted-Credentials-From-Model to master September 10, 2024 12:25
…Khan/Remove-Unencrypted-SAP-Client-Credentials
@MueezKhan246
Copy link
Contributor Author

Thumb of trust 👍 I'm sure you've double checked that it's not being used somewhere else now.

checked again to be sure..

@MueezKhan246 MueezKhan246 merged commit 0f361ae into master Sep 10, 2024
13 checks passed
@MueezKhan246 MueezKhan246 deleted the MueezKhan/Remove-Unencrypted-SAP-Client-Credentials branch September 10, 2024 14:03
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.

3 participants