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

Invalid port range #8

Closed
smares opened this issue Sep 16, 2024 · 2 comments
Closed

Invalid port range #8

smares opened this issue Sep 16, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@smares
Copy link

smares commented Sep 16, 2024

file:///dclint/dist/util/service-ports-parser.js:21
        throw new Error('Invalid port range');
              ^

Error: Invalid port range
    at parsePortsRange (file:///dclint/dist/util/service-ports-parser.js:21:15)
    at file:///dclint/dist/rules/no-duplicate-exported-ports-rule.js:37:42
    at Array.forEach (<anonymous>)
    at file:///dclint/dist/rules/no-duplicate-exported-ports-rule.js:35:25
    at Array.forEach (<anonymous>)
    at NoDuplicateExportedPortsRule.check (file:///dclint/dist/rules/no-duplicate-exported-ports-rule.js:25:24)
    at file:///dclint/dist/linter/linter.js:24:39
    at Array.forEach (<anonymous>)
    at DCLinter.lintContent (file:///dclint/dist/linter/linter.js:23:20)
    at file:///dclint/dist/linter/linter.js:100:43

when using on docker-compose.yml files with environment variable as port mapping.

@zavoloklom
Copy link
Owner

@smares Could you please provide a sample of docker-compose file that cause this error?

@zavoloklom zavoloklom self-assigned this Sep 19, 2024
@zavoloklom zavoloklom added the bug Something isn't working label Sep 19, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
zavoloklom added a commit that referenced this issue Sep 20, 2024
@zavoloklom
Copy link
Owner

@smares fixed in v1.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants