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

Extends TextMapGetter with GetAll() method, implement usage in W3CBaggagePropagator #6852

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Extends TextMapGetter with GetAll() method, implement usage in W3CBaggagePropagator #6852

wants to merge 4 commits into from

Commits on Nov 15, 2024

  1. adds getList() method to TextMapGetter, with default method

    @jamesmoessis
    jamesmoessis committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    35af206 View commit details
    Browse the repository at this point in the history

  2. changes W3CBaggagePropagator to handle multiple headers for baggage, … 

    …using new getList() method on the TextMapGetter. Includes tests.
    @jamesmoessis
    jamesmoessis committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    560d729 View commit details
    Browse the repository at this point in the history

  3. fix checkstyle

    @jamesmoessis
    jamesmoessis committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    6f13bfb View commit details
    Browse the repository at this point in the history

  4. refactors the PR so that we have a separate internal ExtendedTextMapG… 

    …etter, as per review.

Also changes the return type of the method to Iterator<String> as per review, and renames the method to getAll(). Tests are also adjusted
    @jamesmoessis
    jamesmoessis committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    3cd5429 View commit details
    Browse the repository at this point in the history