Skip to content

feat: enhance Docker image name validation and update dependencies#42

Merged
ArmanTaheriGhaleTaki merged 1 commit into
403unlocker:mainfrom
yazdanbhd:main
Aug 25, 2025
Merged

feat: enhance Docker image name validation and update dependencies#42
ArmanTaheriGhaleTaki merged 1 commit into
403unlocker:mainfrom
yazdanbhd:main

Conversation

@yazdanbhd
Copy link
Copy Markdown
Member

Issue: #41

Description:
Replaced custom regex-based Docker image validation with the official kubectl approach using github.com/distribution/reference package.

Changes:

  • Added github.com/distribution/reference dependency
  • Updated DockerImageValidator function to use reference.ReferenceRegexp.MatchString()
  • Removed custom regex pattern and unused imports
  • Updated test cases to match new validation behavior
  • Fixed CLI error handling for validation failures

Files modified:

  • internal/docker/function.go
  • internal/docker/fucntion_test.go
  • cmd/cli/fastdocker.go
  • cmd/cli/root.go
  • go.mod (dependency added)

Testing:

  • All tests pass
  • CLI validation working correctly
  • Build successful

@ArmanTaheriGhaleTaki ArmanTaheriGhaleTaki added the enhancement New feature or request label Aug 21, 2025
@yazdanbhd yazdanbhd force-pushed the main branch 2 times, most recently from ee264c6 to 839dd51 Compare August 25, 2025 18:01
- Added new validation logc using github.com/distribution/reference.
- Improved test cases for Docker image name validation.
@ArmanTaheriGhaleTaki ArmanTaheriGhaleTaki merged commit 644ed76 into 403unlocker:main Aug 25, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants