Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@achanda
Copy link
Contributor

@achanda achanda commented Aug 30, 2016

This subcommand was missing insecure options
Closes #3132

cc @Quentin-M

This subcommand was missing insecure options

Closes rkt#3132
@ghost
Copy link

ghost commented Aug 30, 2016

Can one of the admins verify this patch?

@lucab
Copy link
Member

lucab commented Aug 30, 2016

ok to test

@lucab
Copy link
Member

lucab commented Aug 30, 2016

Thanks for the timely patch @achanda!

In order to check that everything was fine, I wrote some tests for this as part of #3139. Your PR looks fine as-is, and it can be merged once the tests on the other one are green. LGTM.

@lucab lucab added this to the v1.14.0 milestone Aug 30, 2016
@lucab
Copy link
Member

lucab commented Aug 30, 2016

Tests are green, merging.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants