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

디바이스 등록 url param에서 body로 변경 #82

Open
bekker opened this issue Jun 10, 2018 · 0 comments
Open

디바이스 등록 url param에서 body로 변경 #82

bekker opened this issue Jun 10, 2018 · 0 comments
Labels
enhancement: API v2

Comments

@bekker
Copy link
Contributor

bekker commented Jun 10, 2018 •
edited
Loading

상황

(device id는 예제)
[09/Jun/2018:21:59:52 +0900] "POST /v1/user/device/abcdPiyh0O0%3AAPA91bH4W9bLOgk7gPXJP-yni2tMnMoFhWwS5RuITV6rJHiuHUm4jABy5FheUiZzZasw7uSlyepmkTXGpRGOotrLvSl0jmPulDyfs1GuDN9xmtvChrqcNmJe33cN5jsbTWtjC4MWEGja HTTP/2.0" 500 242 "-" "okhttp/2.3.0"
  • device id가 base64인줄 알았는데, 아님..!!
  • 안드로이드 클라의 okhttp의 경우, : 글자가 %3A로 인코딩됨
  • 현재는 express가 자동으로 decoding을 수행하여 잘 작동하지만
  • 장기적으로 param이 아닌 body를 사용하도록 변경해야 하는 것이 바람직
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement: API v2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bekker