Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding ocmstub back in as an EFSS #142

Merged
merged 15 commits into from
Aug 30, 2024
Merged

Adding ocmstub back in as an EFSS #142

merged 15 commits into from
Aug 30, 2024

Conversation

michielbdejong
Copy link
Member

@michielbdejong michielbdejong commented Aug 28, 2024

docker build --build-arg CACHEBUST="default" --file ./docker/dockerfiles/ocmstub.Dockerfile --tag pondersource/dev-stock-ocmstub:1.0 docker/
./dev/ocm-test-suite.sh login ocmstub 1.0 dev electron
docker logs ocmstub1.docker
 6 | const SERVER_NAME = process.env.HOST || "meshdir";
 7 | const SERVER_PORT = process.env.PORT || 443;
 8 | const SERVER_HOST = `${SERVER_NAME}.docker`;
 9 | 
10 | const HTTPS_OPTIONS = {
11 |     key: fs.readFileSync(`/tls/${SERVER_NAME}.key`), cert: fs.readFileSync(`/tls/${SERVER_NAME}.crt`)
                 ^
ENOENT: No such file or directory
   errno: -2
 syscall: "open"
   path: "/tls/ocmstub1.key"

      at /ocmstub/index.js:11:13

Bun v1.1.26 (Linux x64 baseline)

@michielbdejong michielbdejong marked this pull request as draft August 30, 2024 08:19
@michielbdejong
Copy link
Member Author

docker run -it --network=testnet --name=ocmstub1.docker -e HOST=ocmstub1 pondersource/dev-stock-ocmstub:1.0 /bin/bash
-> node stub.js
docker run -it --name=tester --network=testnet ubuntu /bin/bash
-> apt update
-> apt install -y curl
->  curl -k https://ocmstub1.docker

That shows a 'yes' html page. So why does the login script say port 443 is not open?

@michielbdejong
Copy link
Member Author

OK that works now!

@michielbdejong michielbdejong marked this pull request as ready for review August 30, 2024 08:55
@michielbdejong
Copy link
Member Author

I'll merge this and follow up with any improvements necessary

@michielbdejong michielbdejong merged commit b1f7770 into main Aug 30, 2024
29 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant