8000 Additional gcp fetchers by maxhov · Pull Request #123 · Blazebit/blaze-query · GitHub
[go: up one dir, main page]

Skip to content
< 8000 /div>

Additional gcp fetchers - #123

Merged
maxhov merged 38 commits into
mainfrom
additional_gcp_fetchers
Mar 24, 2026
Merged

maxhov merged 38 commits into
mainfrom
additional_gcp_fetchers

Conversation

@maxhov
@maxhov maxhov commented Mar 24, 2026
Copy link
Copy Markdown
Collaborator

No description provided.

mjsprengers and others added 30 commits October 30, 2024 09:21
mjsprengers and others added 6 commits March 18, 2026 15:59
- Add null guards on HttpResponseException.getContent() in catch blocks
  to prevent NPE from destroying the original PermissionDeniedException
  (FirewallRule, Instance, GkeCluster, KmsCryptoKey data fetchers)
- Add null guards on IAM error details chain (getErrorDetails/getErrorInfo)
  and extract isServiceDisabled() helper (Policy, Role, ServiceAccount)
- Narrow catch(Exception) to catch(IOException) in base fetchers to avoid
  masking programming errors (Asset, Folder, Organization, Project)
- Fix DNS fetcher overbroad 403 catch to check accessNotConfigured reason
- Add WARNING logging to all silent continue blocks for API-disabled skips
- Move IAM catch blocks inside project loops so one disabled project
  doesn't skip all remaining projects for that credentials provider
- Add pagination to Workspace Directory fetchers (User, Group, Role,
  RoleAssignment, Member) and Drive fetcher to avoid silent data truncation
- Wrap SqlInstanceDataFetcher NetHttpTransport in try-with-resources
- Add empty IDs guard in WorkItemDataFetcher
- Log DateTimeParseException in WorkItem instead of silently returning null
- Add PermissionDeniedException handling to BucketDataFetcher
- Fix DriveDataFetcher NPE on getDetails() null chain
- Remove commented-out debug code in GCP IAM RoleDataFetcher
@maxhov
maxhov merged commit 6b9d688 into main Mar 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0