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のマージアルゴリズムを独自の関数（ 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の長さが次のようになっていることを確認します。確かに1であり、2ではありません。

evt.innerEvent.CorrelationIDsはこのように重複排除されませんが、変数名は別として、一般的には見栄えがします。

@rvolosatovsのマージアルゴリズムを使用することをお@ pgalic96の提案されたマップ実装とマージ/重複排除すると、効率が大幅に低下し、ソートされていないIDも返されます。

@ pgalic96ステータスを更新できますか？

今週はまだ取り組んでおらず、明日試験後に出社します。 明日それを完成させようとします。

@htdvisserの提案に従ってコードを変更します。