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

wipes profile section when bmx write --use-credential-process is ran #447

Merged
merged 4 commits into from
Jun 6, 2024

Conversation

jkomonen
Copy link
Contributor

@jkomonen jkomonen commented Jun 4, 2024

Why

If there's a profile with the same name in the credentials and config file, it will default to the credentials file, so you will eventually get invalid credentials

Ticket

https://desire2learn.atlassian.net/browse/VUL-329

@jkomonen jkomonen requested a review from a team as a code owner June 4, 2024 21:09
@github-actions github-actions bot added language/csharp size/XS A very small PR - good job! labels Jun 4, 2024
Copy link
Contributor

github-actions bot commented Jun 4, 2024

PR #448 created to format .cs files.

src/D2L.Bmx/WriteHandler.cs Outdated Show resolved Hide resolved
var defaultCredentialsFile = parser.ReadFile( SharedCredentialsFile.DefaultFilePath );
if (defaultCredentialsFile.Sections.ContainsSection( profile ) ) {
defaultCredentialsFile.Sections.RemoveSection( profile );
parser.WriteFile(SharedCredentialsFile.DefaultFilePath, defaultCredentialsFile);
Copy link
Member

Choose a reason for hiding this comment

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

Formatting looks off here.
Please check the auto formatting setup in your IDE and make sure it respects editorconfig. Let us know if you need any help with that.

Copy link
Contributor

github-actions bot commented Jun 5, 2024

PR #449 created to format .cs files.

Copy link
Member

@cfbao cfbao left a comment

Choose a reason for hiding this comment

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

Code change looks good, but please merge the auto format PR to fix the formatting.
Still need to look into your IDE's auto format setup.

Copy link
Contributor

github-actions bot commented Jun 6, 2024

PR #450 created to format .cs files.

@jkomonen
Copy link
Contributor Author

jkomonen commented Jun 6, 2024

Code change looks good, but please merge the auto format PR to fix the formatting. Still need to look into your IDE's auto format setup.

Where can I find the documentation to set this up?

@cfbao
Copy link
Member

cfbao commented Jun 6, 2024

Here: https://desire2learn.atlassian.net/wiki/spaces/MaT/pages/1470857332/IDE+setup+and+auto+formatting
It's mentioned in the "set up dev machine" onboarding card.
If you set this up but it isn't working as expected, please reach out.


Aside, please @ people in GitHub when you're "replying" on the main PR conversation thread, because otherwise we won't get a direct notification.
(Replying to review comments doesn't require @'ing people. We always get direct notification for these.)

@jkomonen jkomonen merged commit d64951a into main Jun 6, 2024
9 checks passed
@jkomonen jkomonen deleted the wipe-profile-section branch June 6, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
language/csharp size/XS A very small PR - good job!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants