API classes' constructors using the default KillBillHttpClient constructor cause 404 because of bad url formatting
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par KillBillHttpClient.getURI() et les constructeurs par défaut utilisés par des classes d’API telles que TenantApi. Reproduisez une requête sans la propriété système killbill.url, puis vérifiez que l’URL construite ne contient plus la barre oblique supplémentaire et que la requête ne renvoie pas 404.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
So API classes (TenantApi for instance) have default constructors that call the default constructor of KillBillHttpClient. The latter relies on system properties or fallback values to init some fileds namely the KillBillHttpClient's kbServerUrl field, so if the system property "killbill.url" isn't set, the fallback value "http://127.0.0.1:8080/" is used instead and since each api has a uri(path) looking like this "/1.0/kb/tenants" , so this line
"u = new URI(String.format("%s%s", this.kbServerUrl, url));" in KillBillHttpClient.getURI() is causing the url used in request to look like this "http://127.0.0.1:8080//1.0/kb/tenants" (notice the // after port), this is causing every request to return null (404 not found) ,as a fix when I set the system property killbill.url to "http://127.0.0.1:8080", requests are successful !
#version1.3.8
- Langage dominant
- Java
- Étoiles
- 35
- Forks
- 68
- Merge moyen
- 11 min
- PR mergées (30 j)
- 3
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de killbill/killbill-client-java
-
Migrate away from Joda-Time to java.timePeut-être à nouveau libre @sbrossie l’a pris il y a 230 jours, et aucune pull request n’est ouverte. Ouverteenhancement
killbill/killbill-client-java#144 · 1 commentaire · 1 personne assignée ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
killbill/killbill-client-java#136 · 7 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
killbill/killbill-client-java#135 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
killbill/killbill-client-java#131 · 1 commentaire ·
-
Issue with Get Queue entriesOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
killbill/killbill-client-java#130 ·
Toutes les issues de killbill/killbill-client-java
Issues similaires
-
Upgrade to Commons Codec 1.23.0Ouvertestatus: team-only type: dependency-upgrade
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
spring-projects/spring-boot#52143 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
area/frontend good first issue kind/cooldown
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
kestra-io/kestra#20704 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour