Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bug: Log4j configuration is not compatible with options #370

Closed
Tracked by #426
Totti0135 opened this issue Aug 9, 2023 · 0 comments · Fixed by arextest/arex-common#9, arextest/arex-api#245 or arextest/arex-storage#70
Closed
Tracked by #426

bug: Log4j configuration is not compatible with options #370

Totti0135 opened this issue Aug 9, 2023 · 0 comments · Fixed by arextest/arex-common#9, arextest/arex-api#245 or arextest/arex-storage#70
Assignees
@binyu1005
Labels
bug Something isn't working good first issue Good for newcomers
Milestone
0.5.0

Comments

@Totti0135
Copy link

cbabd9b3c5002d89730caefd11514cbc
现在的写法在以下场景会有问题：
当connectionString带options的时候，collection会解析不出来，比如下面的connectionString：
mongodb://arex:iLoveArex@localhost:27017/arex_storage_db?replicaSet=66950&authSource=admin&journal=true
connectionString就变成了 mongodb://arex:iLoveArex@localhost:27017/arex_storage_db?replicaSet=66950&authSource=admin&journal=true.logs
.logs被加在了最后
那com.arextest.web.api.service.beans.MongoDb4Provider#MongoDb4Provider在new ConnectionString()的时候，com.mongodb.ConnectionString#ConnectionString 在解析的时候就解析不到logs了，会把journal=true.logs当成一个option，正确的应该是
mongodb://arex:iLoveArex@localhost:27017/arex_storage_db.logs?replicaSet=66950&authSource=admin&journal=true
@binyu1005 binyu1005 self-assigned this Aug 9, 2023
@binyu1005 binyu1005 added bug Something isn't working good first issue Good for newcomers labels Aug 9, 2023
@binyu1005 binyu1005 changed the title log4j配置不兼容带options的场景 bug: log4j配置不兼容带options的场景 Aug 14, 2023
@binyu1005 binyu1005 added the 0831 label Aug 21, 2023
@lijing-22 lijing-22 changed the title bug: log4j配置不兼容带options的场景 bug: Log4j configuration is not compatible with options Aug 22, 2023
@mr3 mr3 mentioned this issue Aug 23, 2023
Merged
This was linked to pull requests Aug 25, 2023
fix mongodb log4j exception arextest/arex-api#245
Merged
bug-fix-mongdb-log4j-exception arextest/arex-storage#70
Merged
@binyu1005 binyu1005 mentioned this issue Aug 29, 2023
Closed
19 tasks
@lijing-22 lijing-22 removed the 0.5.0 label Sep 12, 2023
@lijing-22 lijing-22 added this to the 0.5.0 milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@binyu1005 binyu1005

Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
0.5.0
3 participants
@Totti0135 @binyu1005 @lijing-22