Skip to content

Quick unlock: PIN #111

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.

Sign up for GitHub

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

Jump to bottom
Merged
hazcod merged 26 commits into from
Jan 16, 2022
Merged

Quick unlock: PIN #111

hazcod merged 26 commits into from
Jan 16, 2022

Conversation

@msladek
Copy link
Contributor

@msladek msladek commented Jan 15, 2022

msladek added 25 commits January 5, 2022 01:02
@msladek
initial basic encryption and mpw store
a20590b
@msladek
extract VaultAccessData and improve store
3bcc00c
@msladek
rename pin files
92c02be
@msladek
fix generateAndSetDBKey param
ab9a798
@msladek
fix vault_test
f35d863
@msladek
improve store passphrase generation
96d6a03
@msladek
honor TMPDIR env var
5ef604d
@msladek
add non-interactive mode
5f468e1
@msladek
Merge branch 'master' into pin
efe4304
@msladek
Merge branch 'master' into pin
2a5f1f0
@msladek
securestore improvements
682c983
@msladek
improve logging
3a20cae
@msladek
improve main
32db351
@msladek
symlink resolution for generating vault name
d5a4e01
@msladek
args struct
7e47232
@msladek
kdfIterCount as param
9058c1a
@msladek
command dryrun and split initializeStore
18cd020
@msladek
rename pkg to 'unlock'
16249f5
@msladek
change import for rename
b304b2a
@msladek
update README
2a97ea7
@msladek
update docker image to golang:1.17
2352969
@msladek
linting
07f3191
@msladek
parse int with 32-bit bound
9199850
@msladek
rename password var
2f51064
@msladek
rename to credentials
91ae78e
@hazcod
Copy link
Owner

hazcod commented Jan 15, 2022

Looking really good @msladek !
Thank you for your work.

Anything else you'd like to add before releasing?

@msladek
Copy link
Contributor Author

msladek commented Jan 16, 2022 •
edited
Loading

Thanks @hazcod , happy to contribute! It was a great golang learning experience for me :)

Featurewise no, but since I raised the go version to 1.17 in the action Dockerfile we should probably update the modules' min go version before this merge. Thus I opened #112

@hazcod hazcod merged commit 8f3f1be into hazcod:master Jan 16, 2022
@msladek msladek deleted the pin branch January 16, 2022 23:10
@msladek
Copy link
Contributor Author

msladek commented Jan 16, 2022

Thanks for the merges @hazcod .

The release seems to fail for some reason?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@msladek @hazcod