Skip to content

fix: addressing issues in error handling for example script upload #2680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
christhorwarth wants to merge 1 commit into
base: main
Choose a base branch
from
+5 −3
Open

fix: addressing issues in error handling for example script upload #2680

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions examples/workers/script_upload.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@

import os

from cloudflare import Cloudflare, BadRequestError
from cloudflare import Cloudflare, APIStatusError

API_TOKEN = os.environ.get("CLOUDFLARE_API_TOKEN")
if API_TOKEN is None:
Expand Down Expand Up @@ -73,10 +73,11 @@ def main() -> None:
},
files={
# Add main_module file
# Note: Content-Type must be "application/javascript" or "text/javascript"
script_file_name: (
script_file_name,
bytes(script_content, "utf-8"),
"application/javascript+module",
"application/javascript",
)
# Can add other files, such as more modules or source maps
# source_map_file_name: (
Expand All @@ -88,8 +89,9 @@ def main() -> None:
)
print("Script Upload success!")
print(script.to_json(indent=2))
except BadRequestError as err:
except APIStatusError as err:
print("Script Upload failure!")
print(f"Error code: {err.status_code}")
print(err)


Expand Down