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

Improve docs and examples to make users more aware of SQL injections #10

Merged
merged 9 commits into from
Apr 20, 2023

Conversation

almarklein
Copy link
Owner

@almarklein almarklein commented Apr 20, 2023

Done:

  • Clearer names for parameter names and code variables.
  • Explicitly document all function parameters, and mark those that are potentially unsafe.
  • Update all examples that used the unsafe form (even if it was just using a constant).
  • Added a section to the docs on SQL injection.

Also:

  • New black.
  • Update CI.

@almarklein almarklein merged commit 5f7f1b8 into master Apr 20, 2023
@almarklein almarklein deleted the safer branch April 20, 2023 23:32
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.

1 participant