JessYanCoding / JessYanCoding/ProgressManager

Progress Listener can't listen to url with "/" query

Ouverte
#67 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Java
Étoiles
2.7k
Forks
289
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### Problem

Progress Listener cannot listen to request with query that contains "/", because mResponseListeners Hashmap failed to detect item, when requested "/" change into "%2F", so containsKey(key) will return false

ex.

```kotlin
https://localhost/v3.1?type_file=pdf&start_date=2022/01/07&end_date=2022/02/06
```

the start_date and end_date will cause Problem.

#### Already Done

I've already tried to replace "/" before request with "%2F" but it reversed now in the HashMap key it has "/" but the checked key contains "%2F", so it still return false.

#### Potential Problem

Please check on ProgressManager.java on line 259

#### Potential Fix

Implement more complex search algorithm for checking key, or custom check on HashMap key.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.