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

future-vuls upload command doesn't have --upload flag in v0.23.4 #1740

Closed
takaishi opened this issue Sep 24, 2023 · 2 comments
Closed

future-vuls upload command doesn't have --upload flag in v0.23.4 #1740

takaishi opened this issue Sep 24, 2023 · 2 comments
Labels

Comments

@takaishi
Copy link

What did you do? (required. The issue will be closed when not provided.)

I run future-vuls upload command in Github Action workflow, but it fails.

Error: unknown flag: --url
Usage:
  future-vuls upload [flags]

Flags:
      --config string   config file (default is $HOME/.cobra.yaml)
  -g, --group-id int    future vuls group id, ENV: VULS_GROUP_ID
  -h, --help            help for upload
  -s, --stdin           input from stdin. ENV: VULS_STDIN
  -t, --token string    future vuls token
      --uuid string     server uuid. ENV: VULS_SERVER_UUID

Failed to execute command

I checked v0.23.4 and v0.23.3, and I found --url flag is removed in #1721. Is this intended or not?

v0.23.4:

% ./future-vuls upload -h
Upload to FutureVuls

Usage:
  future-vuls upload [flags]

Flags:
      --config string   config file (default is $HOME/.cobra.yaml)
  -g, --group-id int    future vuls group id, ENV: VULS_GROUP_ID
  -h, --help            help for upload
  -s, --stdin           input from stdin. ENV: VULS_STDIN
  -t, --token string    future vuls token
      --uuid string     server uuid. ENV: VULS_SERVER_UUID

v.0.23.3:

% ./future-vuls upload -h
Upload to FutureVuls

Usage:
  future-vuls upload [flags]

Flags:
      --config string   config file (default is $HOME/.cobra.yaml)
  -g, --group-id int    future vuls group id, ENV: VULS_GROUP_ID
  -h, --help            help for upload
  -s, --stdin           input from stdin. ENV: VULS_STDIN
  -t, --token string    future vuls token
      --url string      future vuls upload url
      --uuid string     server uuid. ENV: VULS_SERVER_UUID

What did you expect to happen?

I expect to use future-vuls upload command with --url flag.

What happened instead?

  • Current Output

Please re-run the command using -debug and provide the output below.

Steps to reproduce the behaviour

Configuration (MUST fill this out):

  • Go version (go version):

go version go1.21.1 darwin/arm64

  • Go environment (go env):

  • Vuls environment:

% git rev-parse --short HEAD
70fd968
  • command:
future-vuls upload --stdin --url ${{ env.FVULS_AUTH_URL }} --group-id ${{ env.VULS_SAAS_GROUPID }} --token ${{ env.VULS_SAAS_TOKEN }} --uuid ${{ env.VULS_SAAS_UUID }}
@takaishi takaishi added the bug label Sep 24, 2023
@wadda0714
Copy link
Contributor

With the update from v0.23.3 to v0.23.4, the url flag in the future-vuls upload command has been removed.
In v0.23.4 future-vuls upload, the --url is no longer needed because the url to upload is specified in the code.
The following command is supposed to work correctly.

future-vuls upload --stdin --group-id ${{ env.VULS_SAAS_GROUPID }} --token ${{ env.VULS_SAAS_TOKEN }} --uuid ${{ env.VULS_SAAS_UUID }}

I apologize for forgetting to mention this change in my PR.

@takaishi
Copy link
Author

I understand why --url flag was removed. I fix my command and confirm it works well. Thank you so much!

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

No branches or pull requests

2 participants