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

Add more expiration options & change key size to 256-bit #59

Closed
wants to merge 4 commits into from

Conversation

manvari
Copy link
Contributor

@manvari manvari commented Apr 3, 2014

Resolves issue #58.

@securitygeneration
Copy link

Thanks for this iceTwy. Just FYI, you also need to put the new expiration options in paste.tpl, otherwise when you click the Clone button on the Paste page, it only shows the original expiration options.

@manvari
Copy link
Contributor Author

manvari commented Apr 8, 2014

Fixed in the new commit.

EDIT: the indentation was messed up, fixed in the commit below.

@manvari manvari changed the title Add more expiration options Add more expiration options & change key size to 256-bit Apr 8, 2014
@manvari
Copy link
Contributor Author

manvari commented Apr 8, 2014

Commit e085d1e resolves #62

@manvari
Copy link
Contributor Author

manvari commented Apr 8, 2014

@sametmax

@sametmax
Copy link
Contributor

sametmax commented Apr 8, 2014

I won't merge both changes, they are separated issues.

@sametmax
Copy link
Contributor

sametmax commented Apr 8, 2014

Ok, forget about this comment, I'm just a little off today.

@sametmax
Copy link
Contributor

sametmax commented Apr 8, 2014

No, I stand corrected again, I can't merge both :) Sorry for the confusion. I really need 2 commits on this one, espacially because I want to suggest some changes on the expiration commit. Sorry for making your life hard. Not intended.

@sametmax sametmax closed this Apr 8, 2014
@manvari
Copy link
Contributor Author

manvari commented Apr 8, 2014

Alright, I'll fix it ;) Give me a second.

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.

3 participants