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

Crypto is Not Defined Error #368

Closed
3 tasks done
danielcrabtree opened this issue Oct 4, 2024 · 2 comments · Fixed by #369
Closed
3 tasks done

Crypto is Not Defined Error #368

danielcrabtree opened this issue Oct 4, 2024 · 2 comments · Fixed by #369
Labels

Comments

@danielcrabtree
Copy link

Contributing guidelines

I've found a bug, and:

  • The documentation does not mention anything about my problem
  • There are no open or closed issues that are related to my problem

Description

v3.7.0 gives ::error::crypto is not defined, rolling back to v3.6.1 resolves the issue.

Expected behaviour

v3.7.0 works without any errors.

Actual behaviour

v3.7.0 gives ::error::crypto is not defined.

Repository URL

No response

Workflow run URL

No response

YAML workflow

name: "Deploy Latest"

on:
  push:
    branches: ["develop"]

jobs:
  docker:
    runs-on: ubuntu-latest
    steps:     
      - name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3
        with:
          config-inline: |
            [registry."docker-registry:5000"]
              http = true
              insecure = true

Workflow logs

No response

BuildKit logs

No response

Additional info

No response

@pandalec
Copy link

pandalec commented Oct 4, 2024

Thank you! This drove me crazy. Tried to switch a (working) dind setup from non-TLS to TLS and I always ran into this issue. Downgrading to 3.6.1 finally did it!

@crazy-max
Copy link
Member

Hum maybe on some runners Node.js is built without including support for the node:crypto module: https://nodejs.org/docs/latest-v18.x/api/crypto.html#determining-if-crypto-support-is-unavailable

I will revert this change for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants