Skip to content

Revert "GVFSVerb: Use OAuth credentials by default"#1808

Merged
vdye merged 1 commit into
masterfrom
default-config-off
Mar 14, 2024
Merged

Revert "GVFSVerb: Use OAuth credentials by default"#1808
vdye merged 1 commit into
masterfrom
default-config-off

Conversation

@derrickstolee
Copy link
Copy Markdown
Contributor

@derrickstolee derrickstolee commented Mar 13, 2024

This reverts commit cbd0cef. This commit was introduced in #1803 based on a comment I made recommending "faster adoption" but that was a mistake. When the release was shipped to winget yesterday, this led to breaks in some pipeline environments.

Using this authentication mode by default is breaking some automated scenarios on build agents that use "gvfs clone". Removing the new default is a fast measure to fix these users, while a longer-term consideration for increasing adoption of OAuth can be done at a later date with the proper parties engaged.

This reverts commit abcb73f.

Using this authentication mode by default is breaking some automated
scenarios on build agents that use "gvfs clone". Removing the new
default is a fast measure to fix these users, while a longer-term
consideration for increasing adoption of OAuth can be done at a later
date with the proper parties engaged.
@derrickstolee
Copy link
Copy Markdown
Contributor Author

(Please ignore the confusion between commits cbd0cef (actually merged in the release) and abcb73f (was in my PR, but cherry-picked into #1804).

@vdye vdye merged commit 9ac4791 into master Mar 14, 2024
@vdye vdye deleted the default-config-off branch March 14, 2024 16:11
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