Lorawan-stack: 중복 된 상관 ID

@htdvisser 문제는 이벤트 페이로드의 상관 ID가 컨텍스트의 ID에 추가되는 것 같습니다.

ctx := events.ContextWithCorrelationID(ctx, fmt.Sprintf("gs:uplink:%s", events.NewCorrelationID()))
msg.CorrelationIDs = append(msg.CorrelationIDs, events.CorrelationIDsFromContext(ctx)...)
registerReceiveUplink(ctx, conn.Gateway(), msg)

상관 관계 ID는 ctx 및 msg ( GetCorrelationIDs() 통해)에 모두 있으므로 이벤트에서 중복됩니다.

여기서 고유성을 확인해야합니까?

https://github.com/TheThingsNetwork/lorawan-stack/blob/master/pkg/events/events.go#L148

예, pkg/events/correlation_context.go 의 병합 알고리즘을 자체 func ( mergeCorrelationIDs )로 이동하여 pkg/events/events.go .

    if cids := data.GetCorrelationIDs(); len(cids) > 0 {
        cids = append(cids[:0:0], cids...)
        sort.Strings(cids)
        evt.innerEvent.CorrelationIDs = mergeCorrelationIDs(evt.innerEvent.CorrelationIDs, cids)
    }

evt.innerEvent.CorrelationIDs 을 정렬해야한다는 메모를 추가해야합니다 (컨텍스트의 상관 관계 ID가 정렬되기 때문입니다).

@ pgalic96 상태는 무엇입니까?

물건으로 바빠서 다음과 같이 고유성을 확인했습니다.

if data, ok := data.(interface{ GetCorrelationIDs() []string }); ok {
        mapCorrelationIds := make(map[string]struct{})
        for _, correlationID := range evt.innerEvent.CorrelationIDs {
            mapCorrelationIds[correlationID] = struct{}{}
        }
        for _, correlationID := range data.GetCorrelationIDs() {
            if _, ok := mapCorrelationIds[correlationID]; !ok {
                evt.innerEvent.CorrelationIDs = append(evt.innerEvent.CorrelationIDs, correlationID)
            }
        }
    }

아직 테스트를 작성하지 않았지만 TestCorrelationID 로 이벤트를 생성 한 다음 페이로드에 동일한 TestCorrelationID 을 제공 한 다음 메시지의 CorrelationID 길이가 다음과 같은지 확인합니다. 실제로 2가 아니라 1입니다.

evt.innerEvent.CorrelationIDs 는 이러한 방식으로 중복 제거되지 않지만 변수 이름을 제외하면 일반적으로보기 좋습니다.

@rvolosatovs 의 병합 알고리즘을 사용하는 것을 선호합니다 (내 의견 참조). 상관 관계 ID를 병합하기위한 효율적인 구현을 만드는 데 이미 시간을 보냈으므로 훨씬 덜 효율적이고 정렬되지 않은 ID를 반환하는 @ pgalic96 의 제안 된 맵 구현과 병합 /

@ pgalic96 상태 업데이트를 줄 수 있습니까?

이번 주에는 아직 작업하지 않았고 내일 시험 후에 출근했습니다. 내일 마무리하려고 노력할 것입니다.

@htdvisser 제안에 따라 코드를 변경