spring-boot-starter icon indicating copy to clipboard operation
spring-boot-starter copied to clipboard

Dynamically determining the `ContentType` based on the current Protocol

Open bobaikato opened this issue 11 months ago • 5 comments

My Concern:

The documentation https://docs.arangodb.com/stable/develop/integrations/spring-data-arangodb/#configuration emphasizes the need to override the ArangoConfiguration#contentType() method when configuring the driver for a protocol with VPACK content type (e.g., VST, HTTP_VPACK, or HTTP2_VPACK).

My Recommendation:

It is unwarranted for users to explicitly override the content type when opting for protocols like VST, HTTP_VPACK, or HTTP2_VPACK. Instead, this information must be seamlessly retrieved from the existing configuration. Moreover, the obligatory override of the ArangoConfiguration unnecessarily compels users to also override the database and ArangoDB.Builder methods, a requirement that can be deemed superfluous.

bobaikato avatar Mar 10 '24 15:03 bobaikato

Fresh avocado detected! Welcome and thank you for your contribution @bobaikato. My avocado-loving overlords have decreed a signed CLA is required for PRs. Please see https://github.com/arangodb/arangodb/blob/devel/CONTRIBUTING.md file to learn more or ask @cw00dw0rd if you have issues.

cla-bot[bot] avatar Mar 10 '24 15:03 cla-bot[bot]

@cw00dw0rd Where can I sign the CLA?

bobaikato avatar Mar 10 '24 15:03 bobaikato

The CLA link is currently broken, see https://github.com/arangodb/arangodb/pull/20737

The correct link should be: https://arangodb.com/wp-content/uploads/2023/09/Download-Contributor-License-Agreement.pdf

rashtao avatar Mar 22 '24 09:03 rashtao

@rashtao I'm supposed to print and sign the document, right? where will it be sent to?

bobaikato avatar Mar 28 '24 07:03 bobaikato

https://arangodb.com/community/#contribute

Just fill in the form, sign and send a scanned copy over to [email protected] or via fax +49-221-2722999-88

rashtao avatar Apr 02 '24 08:04 rashtao

@bobaikato

Sorry, this has been automatically closed as I switched the main branch from master to main, and deleted the master branch.

Please reopen against the main branch. Also please confirm whether you signed the CLA above.

Thank you!

rashtao avatar Jun 19 '24 12:06 rashtao

Hey! No problem, I'm happy to open it. Could you possibly send an electronic file I can sign? It's challenging to remember to print and sign.

bobaikato avatar Jun 19 '24 12:06 bobaikato

I've sent the form.

bobaikato avatar Jul 25 '24 12:07 bobaikato

@rashtao CLA received

fceller avatar Jul 30 '24 07:07 fceller