Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

Separate protected fields (#10) #26

Merged
merged 4 commits into from
Apr 11, 2018
Merged

Conversation

kujyp
Copy link
Contributor

@kujyp kujyp commented Apr 9, 2018

No description provided.

@kujyp kujyp requested a review from harrydrippin April 9, 2018 10:31
@kujyp
Copy link
Contributor Author

kujyp commented Apr 9, 2018

verification headers에 secretkey = “~~“로 입력하면 manager_name’, ‘manager_email’, ‘manager_phone’ 필드를 추가해서줍니다.

이 방식으로 구현해보니,

  1. secretkey가 틀렸을경우 400 bad request에러를 줘야할지, protected_fields를 제외한 값만 줄지 혼동이오네요. 현재는 후자입니다.
  2. 이런방식이라면 protected_field를 요쳥하는 endpoint를 별도로 두고 만드는게 나을까 싶습니다.
  3. 2주전 회의때 언급한 내용인데 Protected_field에 “hided”를 채워서 돌려주는부분. 아직 구현안했습니다. 급한 내용인 강연신청 쪽부터 하려고합니다.

2주전 회의에 언급된내용으로
Unauthorized request도
field = "Hided"로 필드 추가필요해보임.

@harrydrippin
Copy link
Member

Bad Request를 주는 쪽이 나아보입니다. Secret Key를 넣었는데도 실패했다면 정보 탈취 시도로 봐도 무방할 듯 합니다.

@kujyp kujyp merged commit 5616cf2 into development Apr 11, 2018
@kujyp kujyp deleted the kujyp/protected-fields/180409 branch April 11, 2018 08:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants