René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Make usage of http clients (apache, OkHttp, Jetty, Asynch) easier","articleBody":"## Feature request for Cloudinary Java SDK\r\n\r\nGeneral idea: making the settings of http client library to be used easier for consumers of the SDK,\r\nby enabling configuration via `new Cloudinary(...)` constructor.\r\n\r\n## Explain your use case\r\nI want to use plain java http client, instead of Apache one.\r\n\r\n## Describe the problem you’re trying to solve\r\nIn `Cloudinary.loadStrategies()` there's a fixed list of strings with classnames, which a consumer of the SDK can't change.\r\nThis means even if I provide my own strategy, this will not loaded and I still being forced to add dependency for apache http.\r\n(hint: technically: when the class `om.cloudinary.http5.ApiStrategy` is not found, an `UnknownError` is thrown, which is bad)\r\n\r\nAlso this brings me to another point: the constructor of Cloudinary should better accept Configuration object, instead of `Map config`. It's very annoying for me as an SDK user, to provide a Map with configs. \r\nI get no auto-completion on Maps and must find details in either source code or documentation (which is just cumbersome).\r\nSo I don't know immediately what config options are available, that I could e.g. provide HTTP proxy address.\r\n\r\n(Sorry, if I mix two problems here ... I came to these two, when I thought about a solution proposal, below)\r\n\r\n## Do you have a proposed solution?\r\nYES.\r\nThe constructor of Cloudinary.class (or better the Configuration.class) could have some new field `apiStrategy` of type `class\u003cAbstractApiStrategy\u003e`. And another one for the UploadStrategy.\r\nThen, the loadStragies() method will not load by name (which can contain typos), but rather by class(type) directly.\r\n\r\nOn the second problem, adding a new (overloaded) constructor to Cloudinary is easy to add, but that's not sufficient.\r\nIdeally, the Configuration class will provide a convenient Builder function, so users of the SDK are better guided,\r\nand plausibility checks can easier be implemented.\r\n\r\nI'm happy to provide a PR, if this resonates with you.","author":{"url":"https://github.com/nitram509","@type":"Person","name":"nitram509"},"datePublished":"2024-10-03T20:45:30.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/343/cloudinary_java/issues/343"}