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

Don't touch req.URL #38

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Don't touch req.URL #38

merged 1 commit into from
Dec 15, 2023

Conversation

k1LoW
Copy link
Member

@k1LoW k1LoW commented Dec 15, 2023

SSIA.

@k1LoW k1LoW added the bug Something isn't working label Dec 15, 2023
@k1LoW k1LoW self-assigned this Dec 15, 2023
@k1LoW k1LoW changed the title Dont touch req.URL Don't touch req.URL Dec 15, 2023
Copy link
Contributor

Code Metrics Report

main (38eca4d) #38 (f34a392) +/-
Coverage 57.4% 56.5% -0.9%
Code to Test Ratio 1:2.8 1:2.9 +0.0
Test Execution Time 43s 39s -4s
Details
  |                     | main (38eca4d) | #38 (f34a392) |  +/-  |
  |---------------------|----------------|---------------|-------|
- | Coverage            |          57.4% |         56.5% | -0.9% |
  |   Files             |              5 |             5 |     0 |
  |   Lines             |            183 |           186 |    +3 |
  |   Covered           |            105 |           105 |     0 |
+ | Code to Test Ratio  |          1:2.8 |         1:2.9 |  +0.0 |
  |   Code              |            214 |           214 |     0 |
+ |   Test              |            609 |           613 |    +4 |
+ | Test Execution Time |            43s |           39s |   -4s |

Code coverage of files in pull request scope (0.0% → 0.0%)

Files Coverage +/-
testutil/relayer.go 0.0% 0.0%

Reported by octocov

Copy link
Contributor

BenchmarkNGINX-4

main (-) #38 (f34a392) +/-
Number of iterations 10000 10000 0
Nanoseconds per iteration 352491 ns/op 354206 ns/op 1715 ns/op
Bytes allocated per iteration 17425 B/op 17443 B/op 18 B/op
Allocs per iteration 132 allocs/op 133 allocs/op 1 allocs/op
Metadata
main (-) #38 (f34a392)
goos linux linux
goarch amd64 amd64
pkg github.com/k1LoW/rp github.com/k1LoW/rp
cpu AMD EPYC 7763 64-Core Processor AMD EPYC 7763 64-Core Processor

BenchmarkRP-4

main (-) #38 (f34a392) +/-
Number of iterations 10000 10000 0
Nanoseconds per iteration 206701 ns/op 202292 ns/op -4409 ns/op
Bytes allocated per iteration 65323 B/op 65242 B/op -81 B/op
Allocs per iteration 199 allocs/op 199 allocs/op 0 allocs/op
Metadata
main (-) #38 (f34a392)
goos linux linux
goarch amd64 amd64
pkg github.com/k1LoW/rp github.com/k1LoW/rp
cpu AMD EPYC 7763 64-Core Processor AMD EPYC 7763 64-Core Processor

Reported by octocov

@k1LoW k1LoW merged commit da0f049 into main Dec 15, 2023
4 checks passed
@k1LoW k1LoW deleted the dont-touch-req-url branch December 15, 2023 04:13
@github-actions github-actions bot mentioned this pull request Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant