line / jnotebook_reader

:black_joker: An awesome viewer to browse and render Jupyter Notebooks from local, Amazon S3, Google Cloud Storage or MinIO
Apache License 2.0
103 stars 17 forks source link

Unable to use buckets specified as dict #14

Closed ricecooker closed 2 years ago

ricecooker commented 2 years ago

https://github.com/line/jnotebook_reader/blob/b72d3e71560c724fb22d21c62413b205f00cc786/renderer/s3/s3.py#L51-L53

Whenlib/config.py uses a dict to specify buckets, the code above has a typecast exception casting a string to an int even when JNOTEBOOK_READER_S3_BUCKET_NAME is not used. As a result, buckets specified as a dict does not work currently in jnotebook_reader.

mars-martin-line commented 2 years ago

Merged