storj-thirdparty / uplink-python

Python bindings for libuplink
https://storj.io
Apache License 2.0
20 stars 10 forks source link

Bucket name #25

Closed ghost closed 2 weeks ago

ghost commented 1 year ago

Hello .. Does it make sense add lower() to bucket name in project.py ? https://github.com/storj-thirdparty/uplink-python/blob/efba0bc50803599d4570376ad1fd737e0db65b46/uplink_python/project.py#L121

an upper case letter in bucket name causes the "internal error message" or "bucket invalid name" in Uplink CLI

Thank you

kmozurkewich commented 1 year ago

Thanks for the feedback! While that would be a convenience decorator, the bindings should pass-through behavior from the standard libraries, even if errors are generated. We can keep this open for further feedback.