Skip to content

feat: Check for slashes in policy#561

Merged
roneli merged 1 commit into
cloudquery:mainfrom
irmatov:feat/check_for_slashes
Apr 1, 2022
Merged

feat: Check for slashes in policy#561
roneli merged 1 commit into
cloudquery:mainfrom
irmatov:feat/check_for_slashes

Conversation

@irmatov
Copy link
Copy Markdown
Contributor

@irmatov irmatov commented Mar 29, 2022

It is an error if a policy or check label (name) contains a slash.

Differentiate between errors and warnings during loading of config file.

Reverse return values of setupCheckFetchDatabase to comply with Go style
of "error comes last".

It is an error if a policy or check label (name) contains a slash.

Differentiate between errors and warnings during loading of config file.

Reverse return values of setupCheckFetchDatabase to comply with Go style
of "error comes last".
@irmatov irmatov requested review from a team and zagronitay and removed request for a team March 29, 2022 10:56
@github-actions github-actions Bot added the feat label Mar 29, 2022
Comment thread pkg/policy/parser_test.go
@irmatov irmatov self-assigned this Apr 1, 2022
@irmatov irmatov requested a review from zagronitay April 1, 2022 09:03
@bbernays bbernays requested a review from roneli April 1, 2022 11:33
Copy link
Copy Markdown
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@roneli roneli merged commit 3ad0b5b into cloudquery:main Apr 1, 2022
TinLe pushed a commit to TinLe/cloudquery that referenced this pull request Apr 7, 2022
* upstream/main:
  feat: Purge stale data (cloudquery#565)
  feat: Add '--disable-fetch-check' flag (cloudquery#567)
  chore: Sync form .github (cloudquery#569)
  feat: Drift: Improvements (cloudquery#563)
  fix: Added error when connection configuration is not set (cloudquery#476)
  feat: Check for slashes in policy (cloudquery#561)
@irmatov irmatov deleted the feat/check_for_slashes branch June 23, 2022 12:26
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.

3 participants