Skip to content

28/sqli session #29

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Apr 22, 2022
Merged

28/sqli session #29

merged 5 commits into from
Apr 22, 2022

Conversation

jackorcherton
Copy link
Collaborator

@jackorcherton jackorcherton commented Apr 22, 2022 •
edited
Loading

Added my SQLI session

@jackorcherton
Added SQLi Session
292116e 
Added in retrospect of the session I did on 2/03/22
@jackorcherton
added author link
30bbfd4
@jackorcherton jackorcherton added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 22, 2022
@jackorcherton jackorcherton self-assigned this Apr 22, 2022
@jackorcherton jackorcherton changed the title 28/sq li session 28/sqli session Apr 22, 2022
sharkmoos
sharkmoos reviewed Apr 22, 2022
View reviewed changes
Copy link
Contributor

@sharkmoos sharkmoos left a comment

Choose a reason for hiding this comment

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

Looks good. A few minor suggestions but you don't need to take them.

jackorcherton and others added 3 commits April 22, 2022 09:54
@jackorcherton @sharkmoos
Update content/posts/2021_sqli.md
430632b 
Co-authored-by: sharkmoos <muddy117@gmail.com>
@jackorcherton @sharkmoos
changed intercept
6ea779e 
Co-authored-by: sharkmoos <muddy117@gmail.com>
@jackorcherton
Added Boolean SQLi
560168d 
Added Boolean SQLi as suggested by @sharkmoos
@jackorcherton jackorcherton requested a review from sharkmoos April 22, 2022 09:03
sharkmoos
sharkmoos approved these changes Apr 22, 2022
View reviewed changes
Copy link
Contributor

@sharkmoos sharkmoos left a comment

Choose a reason for hiding this comment

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

I'm happy so will approve. I think @Ballo-02 is back in the North, so I'm not sure how much he'll be using his computer or checking github. So I'm happy for you to merge without his approval if you want.

@sharkmoos sharkmoos linked an issue Apr 22, 2022 that may be closed by this pull request
Add SQLi Session #28
Closed
@jackorcherton
Copy link
Collaborator Author

Thanks, only tagged him so that he could see how we do things

@jackorcherton jackorcherton merged commit 281b885 into main Apr 22, 2022
@jackorcherton jackorcherton deleted the 28/SQLiSession branch April 22, 2022 09:49
@Ballo-02
Copy link

I am up north at the minute like @sharkmoos said but will be on my pc soon. I appreciate being cc’d in good to see how you run things, so far so good all organised so I’m happy.

@jackorcherton
Copy link
Collaborator Author

I was going to say @sharkmoos they do have internet up north 😆

@sharkmoos
Copy link
Contributor

I was going to say @sharkmoos they do have internet up north 😆

Wasn't saying he cant access the internet 😆 I was saying he'll probably spend more time outdoors in nature. Something lacking from Coventry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sharkmoos sharkmoos sharkmoos approved these changes

@Ballo-02 Ballo-02 Awaiting requested review from Ballo-02

Assignees

@jackorcherton jackorcherton

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add SQLi Session

3 participants

@jackorcherton @Ballo-02 @sharkmoos