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

Header 필드 대소문자 무시 #67

Open
sanhee opened this issue Dec 1, 2021 · 0 comments · May be fixed by #75
Open

Header 필드 대소문자 무시 #67

sanhee opened this issue Dec 1, 2021 · 0 comments · May be fixed by #75
Labels
기능추가 및 변경

Comments

@sanhee
Copy link
Owner

sanhee commented Dec 1, 2021 •
edited
Loading

  • new TreeMap<>(String.CASE_INSENSITIVE_ORDER);

    • 필드 값의 대소문자 무시하려고 자료구조를 바꿨었음
    • 하지만, 의미 없는 정렬
    • 값을 넣을 때마다 정렬할 이유도 없음

  • Attribute에서 대소문자 구분 하지 않는 로직 구성
sanhee added a commit that referenced this issue Dec 3, 2021
@sanhee
refactor: Request에서 Content-Length를 가져올 수 있도록 수정 (상세내용 o)
a78533d 
#66 #67

+ 헤더(Attribute) 삽입은 일괄 대문자로 하여, 키의 대소문자 관계없이 value를 꺼내올 수 있게 하였음
This was referenced Dec 3, 2021
Closed
Open
sanhee added a commit that referenced this issue Dec 8, 2021
@sanhee
fix: 대소문자를 구분하지 않는 Attributes
76a3235 
#67

기존 들어온 헤더 필드를 강제로 대문자로 변경 후 저장하는 구조였는데, 기존 테스트 케이스가 깨지고, 예상값도 대문자로 해줘야 하는 불편함이 있었습니다.

따라서, Attribute의 맵에 값을 추가할 때 대소문자 관계없이 해당 원소가 존재하는지 맵을 순차적 O(n)으로 비교하고 없을 경우 추가하는 방식으로 로직을 수정했습니다.
@sanhee sanhee linked a pull request Dec 8, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
기능추가 및 변경
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

대소문자를 구분하지 않는 Attributes sanhee/java-was
2 participants
@Dae-Hwa @sanhee