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

securid: disabled by login requirements #170800

Merged
merged 1 commit into from
Apr 10, 2024
Merged

securid: disabled by login requirements #170800

merged 1 commit into from
Apr 10, 2024

Conversation

daeho-ro
Copy link
Member

@daeho-ro daeho-ro commented Apr 6, 2024

Both download and livecheck url are not accessible by login process.


Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

@BrewTestBot BrewTestBot added the missing zap Cask is missing a zap stanza, please add one. label Apr 6, 2024
@Bo98
Copy link
Member

Bo98 commented Apr 7, 2024

@BrewTestBot BrewTestBot removed the missing zap Cask is missing a zap stanza, please add one. label Apr 9, 2024
@daeho-ro daeho-ro changed the title securid: disabled by login requirements securid: update url, livecheck Apr 9, 2024
@daeho-ro
Copy link
Member Author

daeho-ro commented Apr 9, 2024

Now, recover the cask with fixed download url, remove livecheck and change sha256 for not check.

@BrewTestBot BrewTestBot added the missing zap Cask is missing a zap stanza, please add one. label Apr 9, 2024
@daeho-ro
Copy link
Member Author

daeho-ro commented Apr 9, 2024

The link is dead again, I think there is no way to fix the download link either and so disabled.

@daeho-ro daeho-ro changed the title securid: update url, livecheck securid: disabled by login requirements Apr 9, 2024
@bevanjkay
Copy link
Member

Looking at the analytics this has been broken for at least 90 days, and this is the first time someone has tried to fix it, so I'm on board with disabling.

==> Analytics
install: 0 (30 days), 0 (90 days), 98 (365 days)

@bevanjkay bevanjkay added the ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. label Apr 9, 2024
@krehel krehel merged commit ff12862 into Homebrew:master Apr 10, 2024
9 of 11 checks passed
github-actions bot pushed a commit to MPLew-is/homebrew-vscodium that referenced this pull request Apr 10, 2024
…d-livecheck

securid: disabled by login requirements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. missing zap Cask is missing a zap stanza, please add one.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants