Shouldn't check wallet before requests that doesn't require wallet.
- Langage dominant
- Go
- Étoiles
- 30
- Forks
- 32
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Most of GET requests in zcncore like this one: https://github.com/0chain/gosdk/blob/f0d08895c0f1a0abd22f884a8d2fa767c2acbf25/zcncore/wallet_base.go#L1006-L1017
checks for valid wallet, however the wallet is not needed for such a request, should only check sdk config. This affect 0box, for example, since no wallet is configured for it usually.
Related discussion: https://0chain.slack.com/archives/C04EFTQH26T/p1682726300357019
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Le problème pointe vers un fichier spécifique, wallet_base.go, et une plage de lignes (1006-1017) où une requête GET vérifie inutilement la présence d'un portefeuille valide. Commencez par examiner cette fonction et les requêtes GET similaires dans le package zcncore pour comprendre le motif. Déterminez quelles requêtes nécessitent réellement un portefeuille par opposition à la simple configuration du SDK. Vérifiez comment 0box est affecté en raison de l'absence de portefeuille. Exécutez les tests existants pour vous assurer que les modifications ne cassent pas la fonctionnalité.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- go
- Domaine
- api, backend, blockchain
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 45/100