PyGeoFetch 0.1.2__tar.gz → 0.1.4__tar.gz

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pygeofetch
-Version: 0.1.2
+Version: 0.1.4
 Summary: Universal satellite data download pipeline with unified access to 20+ repositories
 Author: pygeofetch Contributors
 License: MIT License
@@ -272,7 +272,7 @@ pygeofetch pipeline schedule weekly-sentinel2.yaml
 
 ## 📋 Documentation
 
-Comprehensive documentation is available at **https://pygeofetch.readthedocs.io**, including:
+Comprehensive documentation is available at **https://appiahkubis14.github.io/pygeofetch-docs/**, including:
 
 - Full CLI reference
 - Provider authentication guides
@@ -284,16 +284,16 @@ Comprehensive documentation is available at **https://pygeofetch.readthedocs.io*
 
 ## 🤝 Contributing
 
-Contributions of all kinds are welcome. See [CONTRIBUTING.md](CONTRIBUTING.md) for full guidelines.
+Contributions of all kinds are welcome. See [CONTRIBUTING.md](https://github.com/appiahkubis14/pygeofetch?tab=contributing-ov-file) for full guidelines.
 
 Good first issues include implementing stub providers to full API integrations, improving test coverage, and adding new post-processing actions.
 
-```bash
+<!-- ```bash
 git clone https://github.com/appiahkubis14/pygeofetch
 cd pygeofetch
 pip install -e ".[dev,all]"
 pytest tests/ -v
-```
+``` -->
 
 ---
 

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/README.md

@@ -189,7 +189,7 @@ pygeofetch pipeline schedule weekly-sentinel2.yaml
 
 ## 📋 Documentation
 
-Comprehensive documentation is available at **https://pygeofetch.readthedocs.io**, including:
+Comprehensive documentation is available at **https://appiahkubis14.github.io/pygeofetch-docs/**, including:
 
 - Full CLI reference
 - Provider authentication guides
@@ -201,16 +201,16 @@ Comprehensive documentation is available at **https://pygeofetch.readthedocs.io*
 
 ## 🤝 Contributing
 
-Contributions of all kinds are welcome. See [CONTRIBUTING.md](CONTRIBUTING.md) for full guidelines.
+Contributions of all kinds are welcome. See [CONTRIBUTING.md](https://github.com/appiahkubis14/pygeofetch?tab=contributing-ov-file) for full guidelines.
 
 Good first issues include implementing stub providers to full API integrations, improving test coverage, and adding new post-processing actions.
 
-```bash
+<!-- ```bash
 git clone https://github.com/appiahkubis14/pygeofetch
 cd pygeofetch
 pip install -e ".[dev,all]"
 pytest tests/ -v
-```
+``` -->
 
 ---
 

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/cli/auth_commands.py

@@ -31,7 +31,6 @@ console = Console()
 def auth() -> None:
     """Manage provider authentication credentials."""
 
-
 @auth.command(name="add")
 @click.argument("provider")
 @click.option("--username", "-u", default=None, help="Username or email.")
@@ -47,7 +46,6 @@ def auth() -> None:
     show_default=True,
     help="Credential storage backend.",
 )
-
 def auth_add(
     provider: str,
     username: str | None,
@@ -60,44 +58,46 @@ def auth_add(
 ) -> None:
     """
     Add credentials for PROVIDER.
-
-    \b
-    Examples:
-      pygeofetch auth add usgs --username samuelyamforo --password USGS@sak@2001server
-      pygeofetch auth add planet --api-key PL_KEY
-      pygeofetch auth add copernicus --username me@example.com
     """
     from pygeofetch.core.authenticator import AuthManager
 
-    creds: dict = {}
-
+    # Initialize credentials dict
+    creds = {}
+    
+    # Handle API key auth
     if api_key:
         creds["api_key"] = api_key
+    # Handle OAuth
     if client_id:
         creds["client_id"] = client_id
     if client_secret:
         creds["client_secret"] = client_secret
     if token:
         creds["token"] = token
-
-    if not creds and not username:
-        username = click.prompt(f"Username for {provider}")
-
-    if username:
+    
+    # Handle username/password auth
+    if username or (not creds):  # If no credentials provided yet, prompt for username
+        if not username:
+            username = click.prompt(f"Username for {provider}")
         creds["username"] = username
-        if not password and not api_key:
+        
+        if not password and not api_key and not token:
             password = click.prompt(f"Password for {provider}", hide_input=True)
         if password:
             creds["password"] = password
-
+    
     if not creds:
         console.print("[red]No credentials provided.[/]")
         sys.exit(1)
-
-    mgr = AuthManager(storage_backend='file')
+    
+    console.print(f"[yellow]Credentials to save: {list(creds.keys())}[/]")  # Debug
+    
+    mgr = AuthManager(storage_backend=store)
     mgr.add_credentials(provider, **creds)
     console.print(f"[green]Credentials for [bold]{provider}[/] saved.[/]")
-
+    
+    
+    
 
 @auth.command(name="login")
 @click.argument("provider")

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/cli/download_commands.py

@@ -128,7 +128,7 @@ def download_run(
         overwrite=overwrite,
     )
 
-    sb = pygeofetch()
+    sb = pygeofetch(auth_backend='file')
 
     # Load data
     if from_search:

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/cli/main.py

@@ -102,7 +102,7 @@ def status(as_json: bool) -> None:
     from pygeofetch.core.engine import pygeofetch
     from pygeofetch.core.cache_manager import CacheManager
 
-    sb = pygeofetch()
+    sb = pygeofetch(auth_backend='file')
     info = sb.status()
     cache_stats = CacheManager().stats()
 

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/cli/search_commands.py

@@ -148,7 +148,7 @@ def search_run(
         timeout_seconds=timeout,
     )
 
-    sb = pygeofetch()
+    sb = pygeofetch(auth_backend='file')
 
     with console.status(f"[cyan]Searching {len(provider_list or [])} provider(s)...[/]"):
         results = sb.search(query, providers=provider_list, use_cache=not no_cache)

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/core/authenticator.py

@@ -1,5 +1,5 @@
 """
-Authentication manager for pygeofetch.
+Authentication manager for PyGeoFetch.
 
 Handles secure credential storage, session caching, and authentication
 flows for all providers. Supports system keyring, encrypted file storage,
@@ -41,7 +41,7 @@ class CredentialStore:
 
     SERVICE_NAME = "pygeofetch"
 
-    def __init__(self, storage_backend: str = "keyring") -> None:
+    def __init__(self, storage_backend: str = "file") -> None:
         self.storage_backend = storage_backend
         self._config_dir = Path.home() / ".pygeofetch"
         self._cred_file = self._config_dir / "credentials.json"
@@ -56,12 +56,19 @@ class CredentialStore:
         self._in_memory[provider] = credentials
 
     def load(self, provider: str) -> Optional[Dict[str, Any]]:
-        """Load credentials for a provider."""
+        """Load credentials for a provider — keyring first, then file fallback."""
         if provider in self._in_memory:
             return self._in_memory[provider]
         if self.storage_backend == "keyring":
-            return self._load_keyring(provider)
-        return self._load_file(provider)
+            result = self._load_keyring(provider)
+            if result:
+                return result
+            # Fall back to file store so credentials saved via --store file
+            # are always visible regardless of which backend is active
+        result = self._load_file(provider)
+        if result:
+            self._in_memory[provider] = result  # cache it
+        return result
 
     def delete(self, provider: str) -> bool:
         """Remove stored credentials for a provider."""
@@ -204,11 +211,11 @@ class AuthManager:
         auth.remove("copernicus")
     """
 
-    def __init__(self, storage_backend: str = "file") -> None:  # Change default to 'file'
+    def __init__(self, storage_backend: str = "file") -> None:
         self.store = CredentialStore(storage_backend=storage_backend)
         self._sessions: Dict[str, AuthSession] = {}
 
-    def add_credentials(
+    def add(
         self,
         provider: str,
         *,
@@ -287,7 +294,6 @@ class AuthManager:
         # Invalidate cached session
         self._sessions.pop(provider, None)
         logger.info(f"Credentials saved for provider {provider!r}")
-        logger.info(cred_data)
 
     def get_credentials(self, provider: str) -> Optional[Credentials]:
         """
@@ -304,6 +310,9 @@ class AuthManager:
             return None
         try:
             from pydantic import SecretStr
+            # Always inject the provider field — it may not be persisted on disk
+            data = dict(data)
+            data.setdefault("provider", provider)
             if "password" in data and isinstance(data["password"], str):
                 data["password"] = SecretStr(data["password"])
             if "api_key" in data and isinstance(data["api_key"], str):
@@ -371,25 +380,20 @@ class AuthManager:
         List all providers with stored credentials.
 
         Returns:
-            List of dicts with provider info and credential types.
+            List of dicts with provider info and session status.
         """
         providers = self.store.list_providers()
         result = []
         for provider in providers:
-            creds = self.get_credentials(provider)
             session = self._sessions.get(provider)
             result.append({
                 "provider": provider,
                 "has_session": session is not None and session.is_valid,
                 "session_expires": session.expires_at.isoformat() if session and session.expires_at else None,
-                "has_username": creds is not None and creds.username is not None,
-                "has_api_key": creds is not None and creds.api_key is not None,
-                "has_token": creds is not None and creds.token is not None,
             })
         return result
 
-
-    def remove_credentials(self, provider: str) -> bool:
+    def remove(self, provider: str) -> bool:
         """
         Remove stored credentials and session for a provider.
 
@@ -425,4 +429,4 @@ class AuthManager:
                     "auth_type": creds.auth_type.value,
                     "username": creds.username,
                 }
-        return result
+        return result

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/core/engine.py

@@ -1,17 +1,17 @@
 """
-pygeofetch main engine.
+PyGeoFetch main engine.
 
-The ``pygeofetch`` class is the single entry point for the Python API.
+The ``PyGeoFetch`` class is the single entry point for the Python API.
 It wires together authentication, search, and download subsystems and
 exposes a clean high-level interface.
 
 Example::
 
     from pathlib import Path
-    from pygeofetch import pygeofetch
+    from pygeofetch import PyGeoFetch
     from pygeofetch.models import SearchQuery
 
-    sb = pygeofetch()
+    sb = PyGeoFetch()
     sb.auth.add("usgs", username="user", password="pass")
 
     results = sb.search(
@@ -47,7 +47,7 @@ class pygeofetch:
     """
     Universal satellite data pipeline.
 
-    ``pygeofetch`` is the top-level API object that provides access
+    ``PyGeoFetch`` is the top-level API object that provides access
     to all subsystems:
 
     - ``sb.auth``     – :class:`~pygeofetch.core.authenticator.AuthManager`
@@ -62,7 +62,7 @@ class pygeofetch:
 
     Example::
 
-        sb = pygeofetch(log_level="INFO")
+        sb = PyGeoFetch(log_level="INFO")
         results = sb.search(SearchQuery(bbox=(-74, 40, -73, 41)))
         sb.download(results, Path("./data/"))
     """
@@ -75,9 +75,10 @@ class pygeofetch:
         cache_ttl: int = 3600,
         max_search_workers: int = 8,
         progress_callback: Optional[Callable] = None,
+        auth_backend: str = "file",
     ) -> None:
         """
-        Initialize pygeofetch.
+        Initialize PyGeoFetch.
 
         Args:
             config_path: Optional path to a custom config YAML file.
@@ -88,9 +89,9 @@ class pygeofetch:
             progress_callback: Optional callable(DownloadProgress) for progress events.
         """
         setup_logging(level=log_level, use_rich=not log_json)
-        logger.debug("Initializing pygeofetch")
+        logger.debug("Initializing PyGeoFetch")
 
-        self.auth = AuthManager()
+        self.auth = AuthManager(storage_backend=auth_backend)
         self.searcher = FederatedSearcher(
             auth_manager=self.auth,
             cache_ttl=cache_ttl,
@@ -104,7 +105,7 @@ class pygeofetch:
         if config_path:
             self._load_config(config_path)
 
-        logger.info("pygeofetch ready")
+        logger.info("PyGeoFetch ready")
 
     # ------------------------------------------------------------------
     # Search
@@ -306,7 +307,7 @@ class pygeofetch:
     def __repr__(self) -> str:
         authed = [item["provider"] for item in self.auth.list()]
         return (
-            f"pygeofetch("
+            f"PyGeoFetch("
             f"authenticated={authed}, "
             f"cache_entries={len(self.searcher.cache._cache)})"
         )

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch/core/searcher.py

@@ -1,5 +1,5 @@
 """
-Federated search engine for pygeofetch.
+Federated search engine for PyGeoFetch.
 
 Provides parallel search across multiple providers with result deduplication,
 relevance scoring, caching, and STAC-compliant output.
@@ -208,20 +208,20 @@ class FederatedSearcher:
 
     def _get_provider(self, provider_id: str) -> Any:
         """Get or create a provider instance with session from auth manager."""
-        if provider_id not in self._provider_instances:
-            from pygeofetch.providers import get_provider
-            prov = get_provider(provider_id)
+        from pygeofetch.providers import get_provider
+        prov = get_provider(provider_id)
 
-            if self.auth_manager and prov.REQUIRES_AUTH:
-                try:
-                    session = self.auth_manager.authenticate(provider_id)
-                    prov.set_session(session)
-                except Exception as exc:
-                    logger.debug(f"Auth for {provider_id} failed: {exc}")
+        if self.auth_manager and prov.REQUIRES_AUTH:
+            # Always re-attempt auth — do not cache unauthenticated providers
+            try:
+                session = self.auth_manager.authenticate(provider_id)
+                prov.set_session(session)
+                logger.debug(f"Authenticated {provider_id} for search")
+            except Exception as exc:
+                logger.warning(f"{provider_id}: authentication failed: {exc}")
+                # Leave provider unauthenticated — require_auth() in search() will raise
 
-            self._provider_instances[provider_id] = prov
-
-        return self._provider_instances[provider_id]
+        return prov
 
     def _get_available_providers(self) -> List[str]:
         """Return providers that have stored credentials or require no auth."""
@@ -338,4 +338,4 @@ class FederatedSearcher:
                 "providers", [{}]
             )[0].get("name", "unknown")
             results.append(SatelliteData.from_stac_item(feature, provider))
-        return results
+        return results

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pygeofetch.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pygeofetch
-Version: 0.1.2
+Version: 0.1.4
 Summary: Universal satellite data download pipeline with unified access to 20+ repositories
 Author: pygeofetch Contributors
 License: MIT License
@@ -272,7 +272,7 @@ pygeofetch pipeline schedule weekly-sentinel2.yaml
 
 ## 📋 Documentation
 
-Comprehensive documentation is available at **https://pygeofetch.readthedocs.io**, including:
+Comprehensive documentation is available at **https://appiahkubis14.github.io/pygeofetch-docs/**, including:
 
 - Full CLI reference
 - Provider authentication guides
@@ -284,16 +284,16 @@ Comprehensive documentation is available at **https://pygeofetch.readthedocs.io*
 
 ## 🤝 Contributing
 
-Contributions of all kinds are welcome. See [CONTRIBUTING.md](CONTRIBUTING.md) for full guidelines.
+Contributions of all kinds are welcome. See [CONTRIBUTING.md](https://github.com/appiahkubis14/pygeofetch?tab=contributing-ov-file) for full guidelines.
 
 Good first issues include implementing stub providers to full API integrations, improving test coverage, and adding new post-processing actions.
 
-```bash
+<!-- ```bash
 git clone https://github.com/appiahkubis14/pygeofetch
 cd pygeofetch
 pip install -e ".[dev,all]"
 pytest tests/ -v
-```
+``` -->
 
 ---
 

{pygeofetch-0.1.2 → pygeofetch-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pygeofetch"
-version = "0.1.2"
+version = "0.1.4"
 description = "Universal satellite data download pipeline with unified access to 20+ repositories"
 readme = "README.md"
 license = { file = "LICENSE" }