bosh-openstack-cpi-release icon indicating copy to clipboard operation
bosh-openstack-cpi-release copied to clipboard

"user_data" needs to be encoded in base64 without newlines

Open ogrand opened this issue 2 years ago • 5 comments

Hello,

Bosh Openstack cpi send a POST request for instance creation, with body content user_data base64 encoded row containing newlines \n every 60 characters:

body: {
  "server": {
    "flavorRef": "s3.small.1",
    "name": "vm-d35f15af-9840-44be-8d95-a2b1e1ad5358",
    "imageRef": "97727138-8334-4188-9b1e-4x558599d99b",
    "user_data": "eyJzZXJ2ZXIiOnsibmFtZSI6InZtLWQzNWYxNWFmLTk4NDAtNDRiZS04ZDk1\nLWEyYjFlMWFkNTM1OCJ9LCJuZXR3b3JrcyI6eyJ0Zi1uZXQtY29hYi1kZXBs\ncy1pb

Recent upgrade (openstack 22.3) is less permissive for base64 decoding and refuses newlines \n inside base64 encoded value.

To be compliant, it will be nice to replace encoding format in /var/vcap/packages/bosh_openstack_cpi/vendor/bundle/ruby/*/gems/fog-openstack-*/lib/fog/openstack/compute/models/server.rb ruby script:

Replace [ascii_userdata].pack('m') with [ascii_userdata].pack('m0') which disable newlines assertion.

Thanks. Olivier

ogrand avatar Sep 27 '22 11:09 ogrand

@ogrand could you create a PR for that?

rkoster avatar Oct 06 '22 15:10 rkoster

Hello Ruben, I create a PR in fog-openstack repository for this issue. Hope It will be merged soon.

Regards. Olivier

ogrand avatar Oct 11 '22 13:10 ogrand

Hello Ruben, do you know if fog-openstack repository included in bosh openstack cpi is always active. Nothing new for my PR : https://github.com/fog/fog-openstack/pull/528

ogrand avatar Oct 19 '22 12:10 ogrand

it seems that the fog-openstack repo is dying as there is no activity since august

ramonskie avatar Feb 02 '23 11:02 ramonskie

@ogrand the upstream pr has been merged could you make a pr so that the the cpi is using the updated version.

ramonskie avatar Jul 03 '23 15:07 ramonskie