Skip to content

containerz/server: hide temp files and add timestamp#63

Open
brianneville wants to merge 1 commit intoopenconfig:masterfrom
brianneville:plugin-tmp-name
Open

containerz/server: hide temp files and add timestamp#63
brianneville wants to merge 1 commit intoopenconfig:masterfrom
brianneville:plugin-tmp-name

Conversation

@brianneville
Copy link
Contributor

Add a timestamp to make it less likely to ever collide with a user-generated name, and make the file hidden to distinguish it from the other plugin files on disk.
Also guard against creating a destination file which has a prefix that could make it look like a temp file

Add a timestamp to make it less likely to ever collide
with a user-generated name, and make the file hidden
to distinguish it from the other plugin files on disk.
Also guard against creating a destination file which
has a prefix that could make it look like a temp file
@coveralls
Copy link

Pull Request Test Coverage Report for Build 21993966361

Details

  • 3 of 7 (42.86%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.07%) to 63.552%

Changes Missing Coverage Covered Lines Changed/Added Lines %
server/deploy.go 3 7 42.86%
Totals Coverage Status
Change from base Build 21969248347: -0.07%
Covered Lines: 1721
Relevant Lines: 2708

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants