Tartarus0zm edited a comment on pull request #16275: URL: https://github.com/apache/flink/pull/16275#issuecomment-868191218
> > Do you think that special characters should be encoded on the UI side? Instead of decoding it on the server again? > > yes and yes. I did a manual check when reviewing #13514 and confirmed that our REST API handles escaped special characters fine. So this is purely a UI issue. @zentol I add some log on UI and REST server, then I found, UI send `0.GroupWindowAggregate(window=[TumblingGroupWindow(%27w$__rowtime__60000)]__properti.watermarkLatency` but REST server received is `0.GroupWindowAggregate(window=%5BTumblingGroupWindow(%252527w$__rowtime__60000)%5D__properti.watermarkLatency` by `RouterHandler`, this has been encoded 3 times, `QueryStringDecoder` decode only once, so this happened. Do you want anything else to discover? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
