search

P3-Core-Dev-Team / P3-Q-A

This repo is for tracking features
Other
0 stars 0 forks source link

ADS-Search - Search form>string value containing single quote - error (Dell) #85

Closed raniangamuthu closed 3 months ago

raniangamuthu commented 4 months ago

image

kilam005 commented 4 months ago

@raniangamuthu
please pass the query synopsis for deeper understanding

@Surinarayanan / @ClementJosh21 Also i believe this is connect with escape character treatment for necessary chars

ShaliniSettu commented 3 months ago

RCA search input which containing single quote is splitting a input values in payload

Fix details replaced the single quotes(') which is coming in payload with double the single quotes('') image image

kilam005 commented 3 months ago

@ShaliniSettu / @ClementJosh21 please check this for multiple select, radio where there is an in query usage in basic and advanced.

There is a chance that the code would not work correctly

raniangamuthu commented 3 months ago

Looks like the fix is not available in current effecTV environment. Will wait for the fix to verify before closure

kilam005 commented 3 months ago

@raniangamuthu this is being worked upon, i have not added Awaitiing release. but we need sample query used for getting some idea to conclude if the popup came up becasue of something else other than quote identifier

Udhayanila8 commented 3 months ago

Tested single quote value in 137 server. Server: 192.168.56.137:4081 image image image image

kilam005 commented 3 months ago

@raniangamuthu Please check this in effectv region and provide closure

kilam005 commented 3 months ago

RCA The ' Single quote is a quote identifier within the SQL engine used by ADS. So it was unable to differentiate key words and input resulting the SQL syntax error detection

Fix Details Code fix was made to escape and handle quote identified.

raniangamuthu commented 3 months ago

Working as expected.