C0rWin

Not strictly related to the aforementioned issue, but I have spotted that `IsDNSName` returns true also for the following ``` govalidator.IsDNSName("192.168.0.") ``` I haven't looked deeper yet, but IMO it...

Details of possible solution is cited in #168

I am not sure the fix actually addresses the path traversal problem. As far as I understand the problem, it could be solved by input sanitation, and I am not...

> > @bhaskarvilles Thanks for the find but it seems the fix does not work. See the error during integration test checks. > > There will be some missing arguments.....

> > > > @bhaskarvilles Thanks for the find but it seems the fix does not work. See the error during integration test checks. > > > > > >...

> Yes, it solves the issue, let me research on the new compilation methods. But I'm sure that it will definitely solves the problem. I'm pretty confident that to solve...

@bhaskarvilles, can you explain the fix, please? Why does the code change address the problem? I think it would be helpful if you could provide a test that manifests the...

> @denyeart Here is the data flow & detailed explanation of this bug. > > **Data flow :** > ## 9 steps in 1 file > > ccaas_builder/cmd/build/main.go 9 steps...

> fair enough, tozd it is. anyone objects? IMO, need to make some MVP to ensure it's indeed drop-in replacement and we won't encounter compatibility issues.

Thanks for providing the fix. Would you also mind adding some unit tests to demonstrate the issue described and to make sure the fix indeed solves the problem? Moreover, to...