Bug 39658 - Тег автора коммита с кавычками, ломает вывод в формате json
Summary: Тег автора коммита с кавычками, ломает вывод в формате json
Status: CLOSED FIXED
Alias: None
Product: Infrastructure
Classification: Infrastructure
Component: girar (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P5 major
Assignee: Dmitry V. Levin
QA Contact: Andrey Cherepanov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-05 19:42 MSK by Sergey Novikov
Modified: 2021-02-06 15:31 MSK (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Novikov 2021-02-05 19:42:33 MSK 
В таске 265849 в "tag_author" попали кавычки:
"tag_author": ""Elena Mishina" <lepata@altlinux.org>"

Это сломало вывод в формате json на git.altlinux.org
http://git.altlinux.org/tasks/265849/info.json?:
{
 "updated": 1612536572,
 "taskid": 265849,
 "shared": false,
 "fail_early": true,
 "test_only": false,
 "repo": "p9",
 "state": "EPERM",
 "try": 1,
 "iter": 1,
 "age": 0,
 "build_time": "2021-02-05T14:49:32",
 "message": "update",
 "owner": "sem",
 "subtasks": {
  "100": {
   "dir": "/gears/d/docs-simply-linux.git",
   "tag_name": "docs-simply-linux/9.09-alt1",
   "tag_id": "33e9203390814de99cbc5cb5fb442a9590ae3cb1",
   "tag_author": ""Elena Mishina" <lepata@altlinux.org>",
   "fetched": "2021-02-05T14:46:00",
   "type": "repo",
   "pkgname": "docs-simply-linux",
   "userid": "sem"
  }
 }
}

$ curl -s http://git.altlinux.org/tasks/265849/info.json? | jq
parse error: Invalid numeric literal at line 20, column 25
Comment 1 Dmitry V. Levin 2021-02-05 20:25:14 MSK 
Как вы предлагаете поступить с этим?
Comment 2 Sergey Novikov 2021-02-05 20:31:47 MSK 
(Ответ для Dmitry V. Levin на комментарий #1)
> Как вы предлагаете поступить с этим?

При формировании json убирать кавычки.
Comment 3 Dmitry V. Levin 2021-02-05 20:55:17 MSK 
(In reply to Sergey Novikov from comment #2)
> (Ответ для Dmitry V. Levin на комментарий #1)
> > Как вы предлагаете поступить с этим?
> 
> При формировании json убирать кавычки.

Только кавычки, больше ничего не помешает?
Comment 4 Sergey Novikov 2021-02-05 21:38:50 MSK 
(Ответ для Dmitry V. Levin на комментарий #3)
> (In reply to Sergey Novikov from comment #2)
> > (Ответ для Dmitry V. Levin на комментарий #1)
> > > Как вы предлагаете поступить с этим?
> > 
> > При формировании json убирать кавычки.
> 
> Только кавычки, больше ничего не помешает?

Да. По идее больше ничего не должно мешать. Если что-то еще помешает, я это увижу и дополнительно сообщу.
Comment 5 Dmitry V. Levin 2021-02-06 00:54:33 MSK 
(In reply to Sergey Novikov from comment #0)
> В таске 265849 в "tag_author" попали кавычки:
> "tag_author": ""Elena Mishina" <lepata@altlinux.org>"

Между прочим, это баг в ключе.
Comment 6 Dmitry V. Levin 2021-02-06 15:31:34 MSK 
(In reply to Sergey Novikov from comment #2)
> (Ответ для Dmitry V. Levin на комментарий #1)
> > Как вы предлагаете поступить с этим?
> 
> При формировании json убирать кавычки.

Should be fixed by commit 33d589f408a00f7bbc53c2a333fdf0b6a31a7764.