Skip to content

fix(logger): fix repository url scheme#717

Merged
ww24 merged 1 commit intomasterfrom
fix-repository-url-scheme
Sep 26, 2023
Merged

fix(logger): fix repository url scheme#717
ww24 merged 1 commit intomasterfrom
fix-repository-url-scheme

Conversation

@ww24
Copy link
Copy Markdown
Owner

@ww24 ww24 commented Sep 26, 2023

No description provided.

@ww24 ww24 self-assigned this Sep 26, 2023
Comment thread logger/report.go

func (r *sourceReference) MarshalLogObject(enc zapcore.ObjectEncoder) error {
enc.AddString("repository", r.Repository)
enc.AddString("repository", strings.Replace(r.Repository, "git://", "https://", 1))
Copy link
Copy Markdown
Owner Author

@ww24 ww24 Sep 26, 2023

Choose a reason for hiding this comment

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

Use https scheme.

@ww24 ww24 enabled auto-merge September 26, 2023 17:40
@ww24 ww24 merged commit 71966c7 into master Sep 26, 2023
@ww24 ww24 deleted the fix-repository-url-scheme branch September 26, 2023 17:44
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.

1 participant