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

Dynamic table reverse lookup search #2

Open
MoMannn opened this issue Oct 15, 2020 · 0 comments
Open

Dynamic table reverse lookup search #2

MoMannn opened this issue Oct 15, 2020 · 0 comments

Comments

@MoMannn
Copy link

MoMannn commented Oct 15, 2020

I was looking and this implementation and I have question regarding dynamic table specifically how reverse lookup works.

If I understand this correctly It first searches static table then goes to the dynamic table where it first checks if name + value are the same if that is not so it checks if only name is the same and if it is and it is not in the static table it returns this value.

But lets say for example that we have a dynamic table like this:

1. "custom-name": "custom-value-1"
2. ":status": "202"
3. "custom-name": "custom-value-2"

And you are encoding header"custom-name": "custom-value-2". In this case since header "custom-name" in not in the static table and because header "custom-name": "custom-value-1" is on the top of dynamic table the encoder will take index 1 and do literal encoding for value instead of using direct match of both name and value at index 3.

Is my understanding of this correct? And if so is this a bug?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant