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

Jump to bottom

fix(x/authz): non utf-8 characters in logs #19923

Merged
merged 9 commits into from
Apr 5, 2024
Merged

fix(x/authz): non utf-8 characters in logs #19923

merged 9 commits into from
Apr 5, 2024

Commits on Apr 2, 2024

  1. write unit test first

    @EmilGeorgiev
    EmilGeorgiev committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    ab6cc27 View commit details
    Browse the repository at this point in the history

  2. implement parsing function that convert grant key to UTF-8 valid string

    @EmilGeorgiev
    EmilGeorgiev committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    fdbe314 View commit details
    Browse the repository at this point in the history

  3. remove unnecessary debug fmt.Println

    @EmilGeorgiev
    EmilGeorgiev committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    c61670e View commit details
    Browse the repository at this point in the history

  4. refactor the function for parsing grsnt key to valid utf-8 string

    @EmilGeorgiev
    EmilGeorgiev committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    5b5e64b View commit details
    Browse the repository at this point in the history

  5. add comments

    @EmilGeorgiev
    EmilGeorgiev committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    a1d24a0 View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2024

  1. for consistency woth other methods print in the logs grenter address,… 

    … grantee address and the msg type
    @EmilGeorgiev
    EmilGeorgiev committed Apr 3, 2024
    Configuration menu
    Copy the full SHA
    a9b1bc5 View commit details
    Browse the repository at this point in the history

  2. remove unnecessary '%s' in the format string

    @EmilGeorgiev
    EmilGeorgiev committed Apr 3, 2024
    Configuration menu
    Copy the full SHA
    ebd7df1 View commit details
    Browse the repository at this point in the history

Commits on Apr 5, 2024

  1. update the changelog.md file

    @EmilGeorgiev
    EmilGeorgiev committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    8b393f0 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into fix-authz-non-utf-8-characters-in-event-logs… 

    …-19874
    @facundomedica
    facundomedica committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    bb306f2 View commit details
    Browse the repository at this point in the history