SkygearIO / py-skygear

Skygear Cloud Code Python SDK
https://skygear.io
Other
8 stars 20 forks source link

py-skygear should not serve asset if ENVVAR `SERVE_STATIC_ASSETS=false` #137

Closed rickmak closed 7 years ago

rickmak commented 7 years ago

Command to run

SERVE_STATIC_ASSETS=false py-skygear

Expected Results

SERVE_STATIC_ASSETS will be false and plugin will serve no static asset

Actual Results

At ConfigArgParse==0.10.0

py-skygear: error: SERVE_STATIC_ASSETS is a flag but is being set to 'false'

If we upgrade ConfigArgParse==0.11.0, there will no error. However, SERVE_STATIC_ASSETS will be true even it is SERVE_STATIC_ASSETS=false. Only missing of ENVVAR will make the setting to be False

Probably need to fix on ConfigArgParse, or avoid using store_true or store_false.

cheungpat commented 7 years ago

Upstream issue https://github.com/bw2/ConfigArgParse/issues/87

cheungpat commented 7 years ago

Problem fixed in dependency. We will the update dependencies when they made a new release in pypi.