Sam

Results 60 comments of Sam

I understand your concerns. OAuth RFC does in fact [recommend not to use localhost](https://datatracker.ietf.org/doc/html/rfc8252#section-7.3), but use `127.0.0.1` instead. However, due to: - Improved developer experience (easy, flexible) - Small attack...

Apologies - I didn't see the lint error before! I can update if you can post the previous lint message 😄 By 'hardcoding' I meant I just removed the list...

Most dependencies should install easily with pip. Psycopg2 is slightly more complex as it requires underlying system libraries to be installed too. There is good documentation about this https://www.psycopg.org/docs/install.html#build-prerequisites Is...

Is work still ongoing on this PR? I have just set up my own dev environment to work on central-backend and central-frontend & documented the process. As a result, I...

Certainly! I made a PR in our repo to more easily show the changes: https://github.com/hotosm/odkcentral/pull/1/files My main goal was to make setting up a dev environment as easy as possible,...

Currently I build a custom image for the central-backend. I have to duplicate the Dockerfile in my repo, as using a setup like this won't work with submodules: ```yaml services:...

Sure thing, the Dockerfile is: ```dockerfile ARG node_version=18 FROM docker.io/bitnami/git:2 as repo ARG ODK_CENTRAL_TAG RUN git clone --depth 1 --branch ${ODK_CENTRAL_TAG} \ "https://github.com/getodk/central.git" \ && cd central && git submodule...

Excellent idea! I thought `deploy.replicas` was a Docker Swarm param, but looks like compose V2 added support 😄 Example config: ```yaml service: central: deploy: replicas: ${BACKEND_REPLICAS:-2} resources: limits: cpus: "0.9"...

+1 it would be very helpful to have pre-built images 👍 @lognaturel would it be useful to have the images multi-architecture, so they work on default AMD64, ARM64 (latest Mac),...

Agree the pre-built images don't necessarily need ARM64 unless asked. If we wanted to consider that in the future, then building the backend [currently fails on ARM64]( https://forum.getodk.org/t/bcrypt-dependency-install-fails-on-arm64-architectures/43992). As for...