Merge pull request #3450 from mozilla/fix/issue-3421

fix(event-broker): allow userAgent to be empty
This commit is contained in:
Ben Bangert 2019-11-22 08:24:20 -08:00 коммит произвёл GitHub
Родитель 47653d3a5e 8ce686a024
Коммит b3630c8b3c
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
2 изменённых файлов: 2 добавлений и 2 удалений

Просмотреть файл

@ -83,7 +83,7 @@ Message Properties:
- `uid`: The userid of the account being that was used to log in. - `uid`: The userid of the account being that was used to log in.
- `email`: The primary email address of the account. - `email`: The primary email address of the account.
- `deviceCount`: The number of active sessions on the user's account. - `deviceCount`: The number of active sessions on the user's account.
- `userAgent`: The user-agent header sent by the user on their login request. - `userAgent`: Optional,the user-agent header sent by the user on their login request.
This event is currently consumed by the email marketing team This event is currently consumed by the email marketing team
in order to manage sync-setup-related email campaigns. in order to manage sync-setup-related email campaigns.

Просмотреть файл

@ -37,7 +37,7 @@ const LOGIN_SCHEMA = joi
timestamp: joi.number().optional(), timestamp: joi.number().optional(),
ts: joi.number().required(), ts: joi.number().required(),
uid: joi.string().required(), uid: joi.string().required(),
userAgent: joi.string().required() userAgent: joi.string().optional()
}) })
.unknown(true) .unknown(true)
.required(); .required();