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

Update Headers.java #1714

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update Headers.java #1714

wants to merge 2 commits into from

Commits on Dec 5, 2023

  1. Update Headers.java 

    asMap() in zuul/message/Headers.java is related to many operations described as expensive (such as toString, hashing). Add a comment to asMap() to better communicate this expensive nature.
    @ctarnold
    ctarnold committed Dec 5, 2023
    Configuration menu
    Copy the full SHA
    e335e77 View commit details
    Browse the repository at this point in the history

Commits on Dec 8, 2023

  1. Update Headers.java 

    Style fix. 1. Added new line before asMap comment. 2. Converted asMap comment to a multiline comment.
    @ctarnold
    ctarnold committed Dec 8, 2023
    Configuration menu
    Copy the full SHA
    584dfa1 View commit details
    Browse the repository at this point in the history