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.

Sign up for GitHub

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

refactor(log): store log in context so it's automatically propagated #1356

Merged
merged 6 commits into from
Mar 19, 2024

refactor: remove `model.Request.Log` in favor of use `Context`

7d8f30f
Select commit
Loading
Failed to load commit list.
Merged

refactor(log): store log in context so it's automatically propagated #1356

refactor: remove `model.Request.Log` in favor of use `Context`
7d8f30f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Mar 19, 2024 in 0s

88.88% of diff hit (target 93.88%)

View this Pull Request on Codecov

88.88% of diff hit (target 93.88%)

Annotations

Check warning on line 128 in resolver/caching_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/caching_resolver.go#L128 

Added line #L128 was not covered by tests

Check warning on line 214 in resolver/caching_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/caching_resolver.go#L214 

Added line #L214 was not covered by tests

Check warning on line 322 in resolver/caching_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/caching_resolver.go#L321-L322 

Added lines #L321 - L322 were not covered by tests

Check warning on line 324 in resolver/caching_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/caching_resolver.go#L324 

Added line #L324 was not covered by tests

Check warning on line 268 in resolver/parallel_best_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/parallel_best_resolver.go#L268 

Added line #L268 was not covered by tests

Check warning on line 270 in resolver/parallel_best_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/parallel_best_resolver.go#L270 

Added line #L270 was not covered by tests

Check warning on line 272 in resolver/parallel_best_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/parallel_best_resolver.go#L272 

Added line #L272 was not covered by tests

Check warning on line 188 in resolver/query_logging_resolver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

resolver/query_logging_resolver.go#L186-L188 

Added lines #L186 - L188 were not covered by tests

Check warning on line 604 in server/server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server.go#L604 

Added line #L604 was not covered by tests

Check warning on line 667 in server/server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server.go#L667 

Added line #L667 was not covered by tests

Check warning on line 156 in server/server_endpoints.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server_endpoints.go#L156 

Added line #L156 was not covered by tests

Check warning on line 173 in server/server_endpoints.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server_endpoints.go#L172-L173 

Added lines #L172 - L173 were not covered by tests

Check warning on line 175 in server/server_endpoints.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server_endpoints.go#L175 

Added line #L175 was not covered by tests

Check warning on line 177 in server/server_endpoints.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/server_endpoints.go#L177 

Added line #L177 was not covered by tests
View more details on Codecov