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

Annotate $app type in front controller #1555

Merged
merged 1 commit into from
Aug 19, 2014
Merged

Conversation

adrolter
Copy link
Contributor

No description provided.

@bobdenotter
Copy link
Member

Despite warnings by @GawainLynch that this change will BREAK EVERYTHING, i'm going to go ahead and merge it in. ;-)

bobdenotter added a commit that referenced this pull request Aug 19, 2014
Annotate $app type in front controller
@bobdenotter bobdenotter merged commit 862b080 into bolt:master Aug 19, 2014
@GwendolenLynch
Copy link
Contributor

Don't say I didn't warn you when there are kittens crying in the street!

@adrolter
Copy link
Contributor Author

See, this is what I like about this project. Always willing to take risks and push the envelope, even when the threat of total meltdown lurks just around the corner.

You guys alright? Pls...

@bobdenotter
Copy link
Member

hf? :-)

@adrolter
Copy link
Contributor Author

I don't know what you're talking about...

Nothing to see here...

@bobdenotter
Copy link
Member

whistles inconspicuously

@adrolter
Copy link
Contributor Author

I can neither confirm nor deny that I accidentally posted a garbage comment to this thread. Hey, what's that over there?

@rarila
Copy link
Contributor

rarila commented Aug 19, 2014

I wonder, if there's a possible backdoor deeply hidden in that pull request. Could be a variation of that dangerous cross site docblock annotation whiskey injec5tion vulnerability threat. I vote for a deep audit! We should hire some experianced security experts for this.

@JarJak JarJak mentioned this pull request Oct 24, 2018
@dadaxr dadaxr mentioned this pull request Feb 27, 2019
@vuchkov vuchkov mentioned this pull request Sep 8, 2019
@vuchkov vuchkov mentioned this pull request Oct 4, 2019
@stryxx stryxx mentioned this pull request Nov 18, 2019
@opernas opernas mentioned this pull request May 7, 2020
@mattmcdonald-uk mattmcdonald-uk mentioned this pull request Jan 28, 2021
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.

None yet

4 participants