Sebastian Ratz

I'd also like to get this into M2 as the current behavior is completely broken with anything other than 100% OS zoom level.

Test failures are unrelated.

I cannot reproduce this. ``` git clone https://github.com/eclipse/lsp4mp.git cd lsp4mp/microprofile.jdt mvn org.eclipse.tycho:tycho-versions-plugin:4.0.9:set-version -DnewVersion=1.2.3-SNAPSHOT ``` works perfectly fine here. Can you run with `-e` and give some information about your environment...

We were also hit by the same issue and I did some analysis: - In swagger-ui 5.17.7 swagger-js was updated from 3.27.3 to 3.27.7. - There, https://github.com/swagger-api/swagger-js/pull/3504 moved from regex...

This issue *is* about the *path*, not the query. A path may also contain `=` characters. E.g., in the example given in the original issue description: `/hello/name={name}` Or, for example,...

I am not using npm, We bundle https://mvnrepository.com/artifact/org.webjars/swagger-ui in our product which is created based on released distributions of swagger-ui.

What's the reason for closing this and not moving to Eclipse 2024-06 for Tycho 4.0.x?

I am not a security expert so do not consider myself qualified to review this code. Besides the security aspects it's - way too many magic numbers and magic strings...

The new diff algorithm introduced in #323 does not really solve the problem. It just got lucky for that one scenario. E.g. compare the following scenarios: 1) 6 lines added...

No objections, I really like this change. Just some technical part: In line 109, we could now do ```diff -return dateFormat.format(entry.getDate()); +return entry.getFormattedDate(); ``` That does internal caching of the...