cloudinary-http5 ignores system properties

Open
#348 2 comments 0 reactions 1 assignee View on GitHub

@adimiz1 is already working on this.

Since Oct 21, 2024.

Assessment

This issue has not been assessed yet.

Description

Describe the bug in a sentence or two.

Previous versions of the library like cloudinary-http45 took system properties into account. cloudinary-http5 doesn't do this anymore, which makes it harder to run the library with different configs.

Issue Type (Can be multiple)

[ ] Build - Can’t install or import the SDK
[ ] Performance - Performance issues
[ x] Behaviour - Functions aren’t working as expected (Such as generate URL)
[ ] Documentation - Inconsistency between the docs and behaviour
[ ] Other (Specify)

Steps to reproduce

Start a VM with http.proxyHost and http.proxyPort parameters. On 1.39.0 this will work, on 2.0.0 the proxy will be ignored.

Versions and Libraries (fill in the version numbers)

Cloudinary Java SDK version - 2.0.0

Repository

Problem is the init method: LINK
The old init method invoked useSystemProperties on the builder. This is missing in 2.0.0

Dominant language
Java
Stars
176
Forks
113
Avg merge
9d 20h
Merged PRs (30d)
2

Getting set up

  • No Dockerfile or Docker Compose file
  • Has a pull request template
  • No contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from cloudinary/cloudinary_java

All issues in cloudinary/cloudinary_java

Similar issues

More Java issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.