-
-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
Use relative paths #1221
Use relative paths #1221
Conversation
…directory of a domain
That's a good one, thank you! I switched it to |
@@ -124,7 +124,7 @@ import { DeluxeUserComponent } from './deluxe-user/deluxe-user.component' | |||
import { AccountingGuard, AdminGuard, LoginGuard, DeluxeGuard } from './app.guard' | |||
|
|||
export function HttpLoaderFactory (http: HttpClient) { | |||
return new TranslateHttpLoader(http, './../assets/i18n/', '.json') | |||
return new TranslateHttpLoader(http, './assets/i18n/', '.json') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still works without navigating one directory up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least for me it works, as the "/#/" is ignored for me, so one directory up is one above the relative directory. I think for root it stays at root, so that's why it wasn't noticed.
Did you try running |
The DOM XSS challenge is unlocked via socket and the API doesn't accept only the dot as servername, so the connection failed. Trying to fix it but I can't test properly locally, as tests keep timing out for me. |
Alright, so the issue was caused by socket.io listening on a default URL path of "socket.io", which needs to be added to the subdirectory. I also had to restart Travis once by closing and re-oping the PR, as a test failed in a different part of the application. |
Woohoo, awesome! |
Great, thanks a lot! Would it be possible to add "Panasonic Information Systems Company Europe" to the package.json contributor list? Would be highly appreciated for the managers! |
I can only add GitHub user names as contributors, not corporations. But if you'd like some swag, I can send stickers and postcards your way. Just mail me your post address. |
As the company paid for the development and testing, it would be great to honor them instead of me. Why is it only possible to add Github usernames? At the moment there are normal names available as well and according to the documentation that's conform to standards. |
Last time I checked corporate acknowledgements are only allowed by OWASP policy on a project Wiki page in the "Acknowledgements" tab. And only for monetary donations, not for contribution of time or effort in any form. |
Also, as you are using a pseudonym account, how would I be able to verify the company you are working for? Using a GitHub account that has ties to e.g. a Panasonic Github organization would have been the easiest way to achieve what you'd like. |
For eg DefectDojo corporate acknowledgements are also possible for contributions (which we used), but this project doesn't have this officially advertised indeed. To verify an identity, I can provide a work email account, so that shouldn't be an issue. Panasonic doesn't have an official Github account and we focus on sponsorship to contribute to open-source projects, as we otherwise can't get it officially signed off. As my commits for work always reference the company it should be clear that those commits are part of work and sponsored by the company. So there is no way for you to willingly add acknowledgements for the company? Then I would probably just edit the PR to reflect it more. |
I'll look into how DefectDojo is doing it and will bring this up with OWASP staff or in the OWASP Leadership meeting tomorrow during GlobalAppSec AMS. I could imagine having a section on the Wiki for corp-powered code contributions... Will let you know then! |
Okay, found it: https://github.com/DefectDojo/django-DefectDojo/blob/master/SPONSORING.md I will definitely not do anything like counting lines of code for points, as LOC is not about the quality of a contribution... 😁 But the way I see it, any company sponsoring their developer's time would be eligible for being mentioned on the Acknowledgements page as if they donated an amount <1000$ - just with name and link but no logo. So, to conclude this: If you name-drop Panasonic somewhere in the PR, then I'll add them to the Ack-page on the Wiki. Also please let me know the exact name and link you'd like in there. |
Added them as "Panasonic Information Systems Company Europe" linking to https://is-c.panasonic.co.jp/en - just let me know if you'd prefer a different link! 👍 |
Wow, that's great, thanks a lot! If you could this link, that'd be great (as it's for our specific branch in Europe): https://application.job.panasonic.eu/data/ruP0pHQvHrGZJKvL/rc.php?nav=jobsearch&custval12=ite&lang=EN&custval11=PBSEU_GER |
Done! Thanks again for your contribution! 💯 |
Hi again, after talking about the whole code-contribution topic with a few other OWASP people I concluded that OWASP would actually need to see some kind official "donation confirmation" from Panasonic to make this an official thing and keep them on the donors list. At the moment I basically took your word for the fact that you did this PR for Panasonic. Also I cannot be sure that your contribution on Panasonic time is not seen by the company as their intellectial property. If that's the case then that would normally be your problem to resolve with them, but by acknowledging them as corporate donors I'm kind of making this my problem. I am no lawyer and honestly don't want to spend my personal time on this kind of stuff. The following would solve this in a quick way:
I'll take down the mention until this is resolved either way. Sorry for the inconvenience, but again: I assume we're both not lawyers, so it's safer for both of us that way... ;-) |
In our company, we would like to run juice-shop as a docker below a certain folder, eg http://example.com/juice-shop (via reverse proxy). This helps us reduce overhead (creating and managing new domain, approvals etc) and helps not being enumerable via DNS.
However, at the moment juice-shop has the root directory hardcoded at a few places, so while the framework supports it, a lot of files are referenced via root.
The minor changes here have been locally tested and allow juice-shop to work in whatever directory it is located in.
This development has been sponsored by Panasonic Information Systems Company Europe.