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

fix: undefined property in access log when handle GRPC request #512

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

reaver-flomesh
Copy link
Collaborator

Description:

Testing done:

Affected area:

Functional Area
New Functionality [ ]
CI System [ ]
CLI Tool [ ]
Certificate Management [ ]
Control Plane [ ]
Demo [ ]
Documentation [ ]
Egress [ ]
Ingress [ ]
Install [ ]
Networking [ ]
Observability [ ]
Performance [ ]
SMI Policy [ ]
Security [ ]
Sidecar Injection [ ]
Tests [ ]
Upgrade [ ]
Other [ ]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project?

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change?

  3. Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)?

@reaver-flomesh reaver-flomesh added area/gateway-api Task/Issue related to GatewayAPI change/pjs Changes that update PJS code kind/bug Something isn't working priority/P0 P0 priority, highest size/XS 1 day labels Dec 11, 2024
@reaver-flomesh reaver-flomesh added this to the v1.5.0 milestone Dec 11, 2024
@reaver-flomesh reaver-flomesh self-assigned this Dec 11, 2024
@reaver-flomesh reaver-flomesh changed the title fix: undefined property when handle GRPC request fix: undefined property in access log when handle GRPC request Dec 11, 2024
Copy link

✅ PR Title Formatted Correctly

The title of this PR has been updated to match the correct format. Thank you!

@reaver-flomesh reaver-flomesh merged commit 327a3fd into main Dec 11, 2024
34 checks passed
@reaver-flomesh reaver-flomesh deleted the bugfix/accesslog branch December 11, 2024 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/gateway-api Task/Issue related to GatewayAPI change/pjs Changes that update PJS code kind/bug Something isn't working priority/P0 P0 priority, highest size/XS 1 day
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant