Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix comment on gravity-db.c #1456

Merged
merged 1 commit into from
Oct 11, 2022
Merged

Fix comment on gravity-db.c #1456

merged 1 commit into from
Oct 11, 2022

Conversation

rdwebdesign
Copy link
Member

By submitting this pull request, I confirm the following (please check boxes, eg [X]) Failure to fill the template will close your PR:

Please submit all pull requests against the development branch. Failure to do so will delay or deny your request

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

How familiar are you with the codebase?:
5


In gravity-db.c, the comment explaining how the wildcard character works when adding audit domains is wrong.

The text says *google.de matches other domains starting in google.de, like abcgoogle.de.
This is clearly wrong, since abcgoogle.de ends with the matching text.
The PR fixes the comment.

This template was created based on the work of udemy-dl.

Signed-off-by: RD WebDesign <[email protected]>
yubiuser
yubiuser previously approved these changes Oct 11, 2022
DL6ER
DL6ER previously approved these changes Oct 11, 2022
@DL6ER DL6ER changed the base branch from master to development October 11, 2022 16:07
@DL6ER DL6ER dismissed stale reviews from yubiuser and themself October 11, 2022 16:07

The base branch was changed.

@DL6ER DL6ER merged commit 26f1e74 into development Oct 11, 2022
@DL6ER DL6ER deleted the fix/comment branch October 11, 2022 16:08
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