Skip to content
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.

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

api: added defaults #551

Merged
merged 5 commits into from
Sep 25, 2024
Merged

api: added defaults #551

merged 5 commits into from
Sep 25, 2024

Conversation

aramikm
Copy link
Contributor

@aramikm aramikm commented Sep 23, 2024

Problem

Related to #491

Solution

  • fixed a previously introduced bug related to link assets

with @pairperson1

Steps to Verify:

  1. run any of the apis
  2. open swagger
  3. try to submit a bigger than supported json

@aramikm aramikm marked this pull request as ready for review September 24, 2024 22:38
transform: true,
}),
);
// TODO: should get replaced with the config
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like config in graph api is not refactored yet!

@@ -23,7 +23,7 @@ x-common-environment: &common-environment

x-content-publishing-env: &content-publishing-env
START_PROCESS: content-publishing-api
FILE_UPLOAD_MAX_SIZE_IN_BYTES: 2000000000
FILE_UPLOAD_MAX_SIZE_IN_BYTES: 500000000
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems like we can only upload files up to 500MB based on current setup and redis

@@ -334,89 +334,6 @@
"health"
]
}
},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for some reason my IDE likes a lot to apply this change.

Copy link
Contributor

@mattheworris mattheworris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Read through changes, everything makes sense to me, nice work.
  • One non-blocking typo
  • Did not test
    🚢 it!

@aramikm aramikm merged commit ef4e554 into main Sep 25, 2024
13 checks passed
@aramikm aramikm deleted the prevent_server_overload branch September 25, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants