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

Fix update script of the Yubico Authenticator and Yubico PIV tool #180

Merged
merged 2 commits into from
May 16, 2023

Conversation

xlammertink
Copy link

@xlammertink xlammertink commented May 9, 2023

Description

As the Yubico Authenticator was not up to date with the latest version (see issue #179), I tried packing and installing it manually. During this process I noticed that the updater script provided an error while updating the release notes, causing the updater to fail. This PR will fix the error.

As mentioned in majkinetor/au#163, it's not possible to perform a search and replace over multiple lines. Therefore I've replaced the actual release notes in the .nuspec file with a URL that points to the release notes and changed the updater script to provide the URL in future updates.

Update: I've also included a fix for issue #181, related to the Yubico PIV tool. Here the updater failed due to the use of wrong quotation in the search and replace function.

Motivation and Context

This will ensure that the updater script will run properly without errors.

How Has this Been Tested?

cd automatic/yubico-authenticator
.\update.ps1
choco pack
choco install yubico-authenticator.6.2.0.nupkg
cd automatic/yubico-piv-tool
.\update.ps1
choco pack
choco install yubico-piv-tool.2.3.1.0.nupkg

Screenshot (if appropriate, usually isn't needed):

Not applicable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this repository (if your change does not follow the style it will likely be rejected - this includes changes in formatting / layout).
  • The changes only affect a single package (not including meta package).

Not applicable:

  • My change requires a change to the documentation (this usually means the notes in the description of a package) and I have updated the documentation accordingly.
  • I have updated the package description and it is less than 4000 characters.
  • The added / modified package passed install / uninstall in the chocolatey test environment.

@xlammertink xlammertink mentioned this pull request May 10, 2023
4 tasks
@xlammertink
Copy link
Author

@pauby, could you take a look at this one and merge if you feel comfortable :)

@xlammertink
Copy link
Author

While checking the status on this PR I noticed another Yubico tool was reported to be broken in issue #181. I've checked the issue and noticed it was related to the quotation in search and replace function in the upgrade script.

I've tested the other Yubico tools as well, but those worked perfectly fine.

@xlammertink xlammertink changed the title Fix Yubico Authenticator update script Fix update script of the Yubico Authenticator and Yubico PIV tool May 16, 2023
@pauby pauby merged commit c1674d8 into pauby:master May 16, 2023
@pauby
Copy link
Owner

pauby commented May 16, 2023

I've merged this now and the latest versions of authenticator and piv-tool have both been pushed. Thanks @xlammertink for the updates. It is appreciated. Apologies for the delay in getting it merged.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants