Title: [QC-670] Add parameter to limit the size of the objects sent to the ccdb by Barthelemy · Pull Request #8013 · AliceO2Group/AliceO2 · GitHub
Open Graph Title: [QC-670] Add parameter to limit the size of the objects sent to the ccdb by Barthelemy · Pull Request #8013 · AliceO2Group/AliceO2
X Title: [QC-670] Add parameter to limit the size of the objects sent to the ccdb by Barthelemy · Pull Request #8013 · AliceO2Group/AliceO2
Description: We have encountered cases when the users would send very large objects to the CCDB and causing havoc later on. We would like to limit the size of the objects sent to the QCDB. I propose to add a parameter to the store* methods specifying such a maximum limit. Doing it on the client side, i.e. in the QC, would mean that we have to do the gymnastic of serializing the object just to know its size although it is done any ways in the CcdbApi. Therfore, I prefer to do it in the ccdbapi to avoid doing it twice in particular given that it is a very expensive operation. I also considered to have the max size as an object member. However one might want to vary the max size of the objects. Last note: the logging of the message when an object is too big might be an issue as it could be repeated many times. On the other hand it is a debug message and it is quite important to be informed that the operation failed. I considered throwing an exception but it would change the interface of these methods and I was not sure it would be wise. What do you think ?
Open Graph Description: We have encountered cases when the users would send very large objects to the CCDB and causing havoc later on. We would like to limit the size of the objects sent to the QCDB. I propose to add a pa...
X Description: We have encountered cases when the users would send very large objects to the CCDB and causing havoc later on. We would like to limit the size of the objects sent to the QCDB. I propose to add a pa...
Opengraph URL: https://github.com/AliceO2Group/AliceO2/pull/8013
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:cee224d6-e5fd-55a2-760c-179cb8bfda05 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 8F9C:468EA:A3E0BC:CDE443:697EFCF5 |
| html-safe-nonce | bf0186383a371d857cddb45f5c6fcede6aff2684436732a25e70c6b91e733042 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4RjlDOjQ2OEVBOkEzRTBCQzpDREU0NDM6Njk3RUZDRjUiLCJ2aXNpdG9yX2lkIjoiNzMyNTk5MTU4MTg4MDYxNDEzMyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | ae891682f3b7611a75505ee003a9f7857f52713d3fe6ea809b5ad82bbe2d2aca |
| hovercard-subject-tag | pull_request:832441189 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/AliceO2Group/AliceO2/pull/8013/files |
| twitter:image | https://avatars.githubusercontent.com/u/4502689?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/4502689?s=400&v=4 |
| og:image:alt | We have encountered cases when the users would send very large objects to the CCDB and causing havoc later on. We would like to limit the size of the objects sent to the QCDB. I propose to add a pa... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 60279d4097367e16897439d16d6bbe4180663db828c666eeed2656988ffe59f6 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/AliceO2Group/AliceO2 git https://github.com/AliceO2Group/AliceO2.git |
| octolytics-dimension-user_id | 9053140 |
| octolytics-dimension-user_login | AliceO2Group |
| octolytics-dimension-repository_id | 24898648 |
| octolytics-dimension-repository_nwo | AliceO2Group/AliceO2 |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 24898648 |
| octolytics-dimension-repository_network_root_nwo | AliceO2Group/AliceO2 |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 7c85641c598ad130c74f7bcc27f58575cac69551 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width