Results 282 comments of Peter A. Jonsson

The tests pass, but I have no idea if the changed code path is exercised by the test suite.

@na9da this is one of the two PRs that together fixes all known security issues in terriajs-server, can you please take a look at this?

@na9da I think https://github.com/yuletide/node-proj4js-defs/pull/2 solves this issue.

@nf-s it takes us 6 minutes to build the docker image in this repository on x86-64 for x86-64. The release workflow also builds images for linux/arm64 through emulation. I can't...

Does the `Dockerfile` in the root of this repository serve any purpose after this PR, or should it be removed?

It looks like my comments about users and permissions have been resolved, is this ready to be merged?

This repository never seems to run the CI until one of you approve it. Can the CI settings be changed so I can make the PR green before you have...

@ljowen @na9da I rebased this after #7272 landed, and since there are now 0 lint warnings, this might be a good time to land this PR?

Latest push fixed cut&paste error that said `java19-sdk, java19-sdk` instead of `java18-sdk, java19-sdk`.

@dongjinleekr I'm not sure if you have seen this, but posting a reminder.