Skip to content

Conversation

@mfridman
Copy link
Member

This PR updates doBrowserLogin to use http:// when TLS is disabled instead of always defaulting to https://.

This ensures browser login works correctly in environments where TLS is not configured.

The intention is to match how we initialize clients elsewhere, e.g.,

if config.TLS == nil {
return buftransport.PrependHTTP(address)
}
return buftransport.PrependHTTPS(address)

@github-actions
Copy link
Contributor

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedDec 31, 2025, 2:23 PM

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.

4 participants