[GitHub] [age] dehowef commented on pull request #790: Age Swift Driver: Check if the graph exists and Readme updated

["dehowef (via GitHub)" <gi...@apache.org>]

dehowef commented on PR #790:
URL: https://github.com/apache/age/pull/790#issuecomment-1511638026

   > > Hey Fahad, when you write a title for a commit, try to make it fit on one line, within the alotted 72 characters. That way your title is brief and readable without it cutting into the description. The description should detail what you did. I'm going to request you make a change to this commit
   > 
   > Updated the title
   
   Please update the commit as well @FahadZaheerfzr 
   
   > > Is passing in the string that way safe? Line 75 in age.swift. Is the input being sanitized to prevent SQL Injections or other attacks?
   > 
   > PostgreCllientKit provides basic protection against SQL injection attacks, and later we can add further security measures
   
   That is good to hear, thank you for checking that


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org