Allow encryption of multiple variables

[sarahhodne]

This allows you to do something like the following:

$ travis encrypt <<EOF
username=random
password=random2
EOF
Please add the following to your .travis.yml file:

  secure: "gfs4uN5PQcp..."
  secure: "asdfsdfDF..."

Pro Tip™: You can add it automatically by running with --add.

Requested in #4

[rkh]

This will break encrypting SSH keys...

[sarahhodne]

You're right. Is there a different way we could do this, or should we reject the feature request?

[rkh]

No, I think in general this is something we want. We could add a flag for splitting it (or for not splitting it).

[rkh]

Maybe we should also not do the data = args.join(" ") then...

[sarahhodne]

No, I think that is wanted. If you do travis encrypt FOO=bar BAR=foobar, you probably want that encrypted as

env:
  - FOO=bar BAR=foobar

is different from

env:
  - FOO=bar
  - BAR=foobar

[drogus]

Also, you can't encrypt too much env vars due to the length limit, so most of the time you will need to keep them split anyway.

[joshk]

env vars which should not create a matrix should go in the global section anyhow

On 18/01/2013, at 12:50 AM, Piotr Sarnacki notifications@github.com wrote:

Also, you can't encrypt too much env vars due to the length limit, so most of the time you will need to keep them split anyway.

—
Reply to this email directly or view it on GitHub.

[rkh]

Yes, --add will add them to env.global, by default, too.

[rkh]

Also, you can't encrypt too much env vars due to the length limit, so most of the time you will need to keep them split anyway.

What's the length limit? Maybe we should check that...

[drogus]

What's the length limit? Maybe we should check that...

I'm not sure, but it's related to restrictions for encrypting using public key. The error is: data too large for key size (OpenSSL::PKey::RSAError)

[rkh]

@henrikhodne what do you think about the commits I added?

[sarahhodne]

�@rkh Looks good to me.