Skip to content

Improvement: Extend serdeContextBase by operationName #1539

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VladimirGorobetsWildix
Copy link

Here’s a clearer and more structured version of your description:


I need to extend serdeContextBase in ServerGenerator.java.

When I make a request to my ssdk, I receive a ValidationException. I want to add metrics for this, but I can't determine which command was executed.

If you add operationName to serdeContextBase, I'll be able to access this information everywhere.

Example:

    @Override
    protected void writeDefaultOutputHeaders(GenerationContext context, OperationShape operation) {
        context.getWriter().write("'x-command-name': ctx.operationName,");
    }

    @Override
    protected void writeDefaultErrorHeaders(GenerationContext context, StructureShape error) {
        context.getWriter().write("'x-command-name': ctx.operationName,");
    }

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@VladimirGorobetsWildix VladimirGorobetsWildix requested a review from a team as a code owner February 28, 2025 18:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant