| Summary: | Error report Web: enhance to support get all configuration in the configuratio files like local.conf/auto.conf/site.conf/toolcfg.conf | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Yocto Project Subprojects] Error Reporting tool | Reporter: | Changqing Li <changqing.li> | ||||||
| Component: | Error Reporting Tool | Assignee: | Changqing Li <changqing.li> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | apoorv sangal <apoorvsangal> | ||||||
| Severity: | enhancement | ||||||||
| Priority: | Medium+ | CC: | changqing.li, mathieu.dubois-briand, raj.khem, randy.macleod | ||||||
| Version: | 2.7 | ||||||||
| Target Milestone: | 5.99 | ||||||||
| Hardware: | x86 | ||||||||
| OS: | Multiple | ||||||||
| Whiteboard: | |||||||||
| OS type for building Yocto: | --- | Type of Regression: | --- | ||||||
| Verified: | Documentation change: | No (bug/feature does not impact docs) | |||||||
| Attachments: |
|
||||||||
|
Description
Changqing Li
2019-04-01 02:06:03 UTC
This was fixed with https://git.openembedded.org/openembedded-core/commit/?id=7adf9707c04d8ef6bcd8d8bda555687f705e6ee6 however, it caused a new problem when local.conf or auto.conf contains BUILDHISTORY_COMMIT_AUTHOR ?= "Khem Raj <raj.khem@gmail.com>" then it confuses, <> as JSON tokens and as a result when send-error-report is called it does not upload the error report and spits out Preparing to send errors to: errors.yoctoproject.org ERROR: HTTP Error 500: OK I can still see the problem. After enabling error report in local.conf Add BUILDHISTORY_COMMIT_AUTHOR ?= "Joe <joe@joe.joe" to conf/local.conf build a package which fails in some task Try to upload the resulting error report % send-error-report /mnt/b/yoe/master/build/tmp/log/error-report/error_report_20210320152535.txt Preparing to send errors to: errors.yoctoproject.org Review information before sending? (y/n): n ERROR: HTTP Error 500: OK Created attachment 4778 [details]
testcase
try to send this error report and issue can be reproduced too.
Created attachment 4779 [details] same error report which works this error report is generated with https://lists.openembedded.org/g/openembedded-core/message/149444?p=,,,20,0,0,0::Created,,report-error,20,2,0,81339399 applied and it can be sent correctly. Update from Changqing via email: There are 3 parts for this defect: 1. oe-core: report-error.bbclass support local.conf -> merged in YP and WR https://git.openembedded.org/openembedded-core/commit/meta/classes?id=a5afd7b1857c0012ab6e3d8bbafa67a96ff9e9e0 merged 2. error-report-web: fix XSS issue -> merged in YP, and I will cherry-pick it to WR and verify if it works. https://git.yoctoproject.org/error-report-web/commit/?id=acae5a6e2c4ceac94736af668397daafd98afdbe 3. error-report-web: support display local.conf and auto.conf in error-report-web -> merged in WR, and I need to resend patch to YP. So, I think we still need this bug open, and I will update the status after all the work all done. Bulk move from 4.99 or 0.00 to 5.99 oe recently add another default configuration file toolcfg.conf, update title to cover 4 default configuration files: local.conf auto.conf site.conf toolcfg.conf. Patch send: https://lists.openembedded.org/g/openembedded-core/topic/119326707 https://lists.yoctoproject.org/g/yocto-patches/topic/119326706 Patch merged: https://bugzilla.yoctoproject.org/show_bug.cgi?id=13252 |