fix to ensure to call requestdone() when requestkey is valid

[sykim-etri]

CB-SP에서 에러를 리턴하는 경우도 왕왕(동일한 이름의 노드그룹 추가 등) 있기 때문에 requestkey 값을 확인한 후 requestDone()을 호출하도록 수정한 PR입니다.
method가 GET인 경우에 한정하여 requestDone()을 호출하는 방안도 있겠습니다.
의견 주시면 반영하도록 하겠습니다.

fix: #1557

[seokho-son]

@sykim-etri 버그 확인 및 개선 감사합니다. :)

해당 사항은

cb-tumblebug/src/core/common/client.go

Line 80 in bdbbbfd

func requestDone(requestKey string) {

함수 내에서 개선하면 좋을 것 같습니다.

• if requestKey != "" 인 경우 함수 내에서 바로 리턴하고,
• count, _ := clientRequestCounter.Load(requestKey) 에 err 에러 핸들링 추가해서, err이 있는 경우 그냥 리턴하면 좋을 것 같습니다.

어떠신지요?

[sykim-etri]

@sykim-etri 버그 확인 및 개선 감사합니다. :)

해당 사항은

cb-tumblebug/src/core/common/client.go

Line 80 in bdbbbfd

func requestDone(requestKey string) {

함수 내에서 개선하면 좋을 것 같습니다.

• if requestKey != "" 인 경우 함수 내에서 바로 리턴하고,
• count, _ := clientRequestCounter.Load(requestKey) 에 err 에러 핸들링 추가해서, err이 있는 경우 그냥 리턴하면 좋을 것 같습니다.

어떠신지요?

방어 코드를 추가하는 측면에서는 바람직할 것 같습니다.

requestKey 생성 시점과의 pair를 생각한다면.. GET인 경우만 requestDone을 호출하도록 수정하고.. 의견주신 방어코드를 추가하는 것은 어떨까요?

[seokho-son]

@sykim-etri 넵! 제안해주신 방식이 더 명확할 것 같습니다. :)

[seokho-son]

/approve