Skip to content

Dev#170

Merged
thibauld merged 2 commits intomainfrom
dev
Jun 12, 2025
Merged

Dev#170
thibauld merged 2 commits intomainfrom
dev

Conversation

@thibauld
Copy link
Collaborator

What?

Please describe what you're trying to accomplish in this PR.

Why?

What problem does this solve? Why is this important? What's the context?

Screenshots (optional)

@thibauld thibauld merged commit cd0a44e into main Jun 12, 2025
2 of 3 checks passed
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: GitHub Secrets Not Forwarded to Docker

Three new secrets (FAIRMINT_MAINNET_USER_ID, FAIRMINT_MAINNET_PARTY_ID, TRANSFER_AGENT_MAINNET_CLIENT_SECRET) are added to the GitHub Actions job environment but are not forwarded to the Docker container via -e flags in the docker run command. Consequently, the application inside the container will not receive these required configuration values, leading to runtime failures or incorrect behavior.

.github/workflows/deploy.prod.yaml#L57-L60

RPC_URL: ${{ secrets.RPC_URL }}
FAIRMINT_MAINNET_USER_ID: ${{ secrets.FAIRMINT_MAINNET_USER_ID }}
FAIRMINT_MAINNET_PARTY_ID: ${{ secrets.FAIRMINT_MAINNET_PARTY_ID }}
TRANSFER_AGENT_MAINNET_CLIENT_SECRET: ${{ secrets.TRANSFER_AGENT_MAINNET_CLIENT_SECRET }}

Fix in Cursor


Was this report helpful? Give feedback by reacting with 👍 or 👎

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