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

I got an error 'unexpected end of JSON input #1928

Closed
sykim-etri opened this issue Nov 18, 2024 · 2 comments · Fixed by #1931
Closed

I got an error 'unexpected end of JSON input #1928

sykim-etri opened this issue Nov 18, 2024 · 2 comments · Fixed by #1931
Assignees
Labels
bug Something isn't working

Comments

@sykim-etri
Copy link
Member

sykim-etri commented Nov 18, 2024

이미 삭제된 것으로 보이는 shared resource에 대해 삭제(refine option)할 때 아래와 같은 에러가 발생합니다. 단순히 로그로만 찍히는 내용이긴 합니다만 특이 사항이 없는 내용에 대해 에러가 출력되는 것으로 보여서 확인을 부탁드립니다.
@yunkon-kim

8:17AM INF src/core/resource/vnet.go:929 > RefineVNet
8:17AM ERR src/core/common/label/label.go:109 > labelInfo: { map[]} error="unexpected end of JSON input"
8:17AM WRN src/core/resource/vnet.go:1071 > error="unexpected end of JSON input"
8:17AM ERR src/core/common/label/label.go:109 > labelInfo: { map[]} error="unexpected end of JSON input"
8:17AM WRN src/core/resource/vnet.go:1071 > error="unexpected end of JSON input"
8:17AM INF src/api/rest/server/middlewares/zerologger.go:59 > request ID=1731917821936507266 Method=DELETE URI=/tumblebug/ns/ns01/resources/vNet/ns01-shared-tencent-ap-seoul?action=refine clientIP=129.254.180.180 latency=1.3538309s status=200
@sykim-etri sykim-etri added the bug Something isn't working label Nov 18, 2024
@yunkon-kim
Copy link
Member

@sykim-etri

버그로 보기는 어려울 것 같고요. 관련하여 의견을 공유드립니다.

Refine은 예외적인/비정상적인 사용으로 인해 발생하는 비정상 Object를 정리하고, 정상적인 Object만을 정제하기 위한 지원하기 위해 추가된 옵션입니다. 그래서 Error 처리 대신, Warning 으로 적용한 상황입니다.

Ref #1808

  1. Spider에서 metadata가 삭제되고, CSP에 리소스가 남아 있는 경우
  2. Spider에 metadata가 있으나, CSP에서 리소스가 삭제된 경우

Error는 label에서 발생하고 있으며, 이는 Global 공통 함수로 Refine만으로 위해 log level 수정을 어려워 보입니다.

@sykim-etri
Copy link
Member Author

sykim-etri commented Nov 18, 2024

@seokho-son
labelInfo의 Unmarshal() 전에 ""를 분류하여 처리하는 것이 낫진 않을까요?
(해당 API의 전반적인 내용을 보진 않고 살짝 보고 단순 의견만 드리는 겁니다.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants