-
Notifications
You must be signed in to change notification settings - Fork 186
UT_SONAR_TEST_REPORTER: wrong/missing encoding information #676
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.
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
Comments
Hi @pesse did you check this on develop branch. A failure message was changed to no longer use a cddata but to convert message by xml for 3.1.2. Wonder if that will help. |
Can you point to the commit? |
It's in PR |
Can't see how this should solve the problem to be honest. |
It will not help when there are special characters in test description for example |
Trouble is - I cant find any valuable info on how to detect current encoding and put it into XML.
|
After some digging around here is what I've found. ResourcesInfo on allowed&valid values for Converting between Oracle and IANA namesLookup of Oracle to IANA on google led me to ora_i18n.map_charset function. Having that function we can now convert Oracle character-set name to IANA name. select utl_i18n.map_charset(value) iana_charser,
value oracle_charset
from v$nls_valid_values x
where parameter = 'CHARACTERSET'; We could now use the below query to get the XML/HTML select utl_i18n.map_charset(value), value from v$nls_parameters where parameter = 'NLS_CHARACTERSET'; The trouble is however that we don't know client-side encoding.
Conclusions?We need to take one of 2 actions:
|
New plan after ongoing discussion in utPLSQL/utPLSQL-cli#78:
I guess the latter two tasks will require to add new param to ut_output_reporter type - might also need additional care in java-api for compatibility. |
Atm the encoding charset of the result-XML of
ut_sonar_test_reporter
is not correct (or missing?)See utPLSQL/utPLSQL-cli#78
We should add charset encoding based on the current session's charset setting.
We should also check other reporters which might be affected from charset mismatch/missing information.
The text was updated successfully, but these errors were encountered: