Lorawan-stack: Повторяющиеся идентификаторы корреляции

@htdvisser проблема

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

Идентификаторы корреляции дублируются в событии, поскольку они оба находятся в 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 необходимо отсортировать (так оно и есть, потому что идентификаторы корреляции в контексте сортируются).

Какой статус у @ 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 в полезной нагрузке, а затем проверять, что длина CorrelationIDs сообщения равна действительно 1, а не 2.

Помимо имени переменной, это обычно выглядит хорошо, хотя evt.innerEvent.CorrelationIDs не дедуплицируются таким образом

Я бы предпочел использовать алгоритм слияния @rvolosatovs (см. Мой комментарий). Мы уже потратили время на создание эффективной реализации для слияния идентификаторов корреляции, поэтому было бы обидно, если бы мы слили / дедуплицировали с предложенной реализацией карты @ pgalic96, которая намного менее эффективна и также возвращает несортированные идентификаторы.

@ pgalic96 можешь дать обновление статуса?

Я еще не работал над этим на этой неделе, приду в офис завтра после экзамена. завтра постараюсь доработать.

Я изменю код в соответствии с предложением @htdvisser .