spin icon indicating copy to clipboard operation
spin copied to clipboard

`spin deploy` with bindle ID "bacongobbler/myapp" or "fermyon.com/myapp" fails

Open bacongobbler opened this issue 2 years ago • 3 comments

This is the same issue as present in https://github.com/deislabs/hippo/pull/723. The bindle "storage ID" and the application name validation regexes differ slightly. Certain bindle IDs are not valid application names. We need to relax this constraint in Hippo, or "sanitize" the bindle ID before submitting it as an application name.

https://github.com/fermyon/spin/blob/fcb88ae6371c932530c03fa653b7eb865eb82aca/src/commands/deploy.rs#L206

bacongobbler avatar May 16 '22 22:05 bacongobbler

https://github.com/deislabs/hippo/blob/f0828b0aa6f2f2777a0ff22616f5bb95e7abc400/src/Application/Apps/Commands/CreateAppCommandValidator.cs#L13-L15

The application name is used as the default domain name.

https://github.com/deislabs/hippo/blob/f0828b0aa6f2f2777a0ff22616f5bb95e7abc400/src/Application/Channels/Commands/CreateChannelCommand.cs#L52-L54

bacongobbler avatar May 16 '22 22:05 bacongobbler

@bacongobbler I'm going through some old issues and found this. Has this been addressed yet, or does it still need attention?

dicej avatar Sep 02 '22 21:09 dicej

@bacongobbler bumping this. It looks from my reading of the Spin code that like the application name remains unsanitised on the client, and that the requirement is still there on the Hippo side, so I'm keeping it alive, but if you have time to comment on priority and next steps, that would be super useful.

itowlson avatar Sep 22 '22 02:09 itowlson