anz-bank / sysl

Sysl (pronounced "sizzle") is a system specification language
https://sysl.io
Apache License 2.0
122 stars 42 forks source link

Add default value when SYSL_PLANTUML is unset #919

Closed ChloePlanet closed 4 years ago

ChloePlanet commented 4 years ago

Current Sysl Version

v0.123.0

Use-cases

export SYSL_PLANTUML=http://www.plantuml.com/plantuml is required to generate diagram at present. Adding a default value to it if SYSL_PLANTUML is unset.

anzdaddy commented 4 years ago

Changing the default to a public service is dangerous, since it will result in accidental leakage of work product from organisations using Sysl internally. A better solution is to keep the default but provide helpful instructions in the error output if localhost:8080 doesn't work.