Skip to content

INTERNAL: Change some errors to INVALID#834

Open
ing-eoking wants to merge 1 commit intonaver:developfrom
ing-eoking:invalid
Open

INTERNAL: Change some errors to INVALID#834
ing-eoking wants to merge 1 commit intonaver:developfrom
ing-eoking:invalid

Conversation

@ing-eoking
Copy link
Copy Markdown
Collaborator

🔗 Related Issue

  • jam2in/arcus-works#710

⌨️ What I did

  • CLIENT_ERROR에서 reconnect가 필요없는 에러 메시지를 INVALID로 변경합니다.
  • 일부 admin 명령어는 INVALID로 변경했습니다.
    • too many prefixes와 관련된 에러
    • ENGINE_EBADVALUE와 관련된 에러
    • shutdown을 제외한 DENIED를 반환하는 에러

@ing-eoking ing-eoking marked this pull request as draft April 17, 2025 01:33
@ing-eoking ing-eoking marked this pull request as ready for review April 17, 2025 01:38
@ing-eoking ing-eoking requested review from jhpark816 and namsic April 17, 2025 01:42
Copy link
Copy Markdown
Collaborator

@namsic namsic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

일부 admin 명령어는 INVALID로 변경했습니다.
shutdown을 제외한 DENIED를 반환하는 에러

shutdown을 제외하는 이유는 무엇인가요?

@ing-eoking
Copy link
Copy Markdown
Collaborator Author

shutdown을 제외하는 이유는 무엇인가요?

통일성을 위해 "DENIED" 응답을 모두 "INVALID"로 변경하려고 했습니다.

그러나 shutdown 명령의 경우, 다른 에러들과는 다르게 특정 상황에서 해당 명령이 거부됨을 나타내기에 유지했습니다.
또한, shutdown은 관리자(admin) 명령이긴 하지만, 이 경우에는 연결을 유지하는 것보다 종료하는 것이 더 적합한 동작이기 때문에 이를 그대로 두었습니다.

@jhpark816
Copy link
Copy Markdown
Collaborator

@ing-eoking
본 PR이 valid한 지 확인해 주세요.

@ing-eoking
Copy link
Copy Markdown
Collaborator Author

@jhpark816
현재는 해당 작업을 재정리가 필요한 사안으로 보고 있습니다.

"CLIENT_ERROR bad value" 응답 메시지의 일부를 "INVALID"로 변경하는 방안에 대해 검토 중이지만,
어떤 상황에서 이를 적용하는 것이 적절할지는 아직 명확하지 않은 상태입니다.

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

Successfully merging this pull request may close these issues.

3 participants