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

Len 0 values logged as 0x in responses #424

Closed
kriztalz opened this issue Feb 18, 2025 · 3 comments · Fixed by #433
Closed

Len 0 values logged as 0x in responses #424

kriztalz opened this issue Feb 18, 2025 · 3 comments · Fixed by #433
Assignees
Labels
bug Something isn't working easy An easy issue to pick up for anyone. good first issue Good for newcomers mentor available A mentor is available to help you through the issue.
Milestone

Comments

@kriztalz
Copy link
Contributor

kriztalz commented Feb 18, 2025

When certain TLS extensions of len 0 are used in the Client Hello of a request, rama returns 0x.

Example for a TLS 1.2 curl request to echo.ramaproxy.org:

"tls": {
             ...
      {
        "data": "0x",
        "id": "NEXT_PROTOCOL_NEGOTIATION (0x3374)"
      },
      {
        "data": [
          "h2",
          "http/1.1"
        ],
        "id": "APPLICATION_LAYER_PROTOCOL_NEGOTIATION (0x0010)"
      },
      {
        "data": "0x",
        "id": "ENCRYPT_THEN_MAC (0x0016)"
      },
      {
        "data": "0x",
        "id": "EXTENDED_MASTER_SECRET (0x0017)"
      }
}
@GlenDC
Copy link
Member

GlenDC commented Feb 18, 2025

Good catch. Is this something you are willing to pickup?

For those that want to pick it up, here are the locations to start your patch journey:

@GlenDC GlenDC added this to the v0.2 milestone Feb 18, 2025
@GlenDC GlenDC added bug Something isn't working good first issue Good for newcomers easy An easy issue to pick up for anyone. mentor available A mentor is available to help you through the issue. labels Feb 18, 2025
@airycanon
Copy link
Contributor

@GlenDC I would like to have a try, could you please assign to me?

@GlenDC
Copy link
Member

GlenDC commented Feb 19, 2025

Sure thing. All yours. Do let me know if you need help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easy An easy issue to pick up for anyone. good first issue Good for newcomers mentor available A mentor is available to help you through the issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants