Log host/ip for access violation

[Commifreak]

Summary

During checking some Host violates local access rules I noticed, that there is no value being logged. This PR adds the affected Host/IP to the Exception.

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[ChristophWurst]

Makes sense

[Commifreak]

Makes sense

I also added the hostName and port to the message.

[skjnldsv]

There were 3 failures:

1) lib\Http\Client\DnsPinMiddlewareTest::testRejectIPv4
Failed asserting that exception message 'Host "192.168.0.1" (www.example.com:80) violates local access rules' contains 'Host violates local access rules'.

2) lib\Http\Client\DnsPinMiddlewareTest::testRejectIPv6
Failed asserting that exception message 'Host "fd12:3456:789a:1::1" (ipv6.example.com:80) violates local access rules' contains 'Host violates local access rules'.

3) lib\Http\Client\DnsPinMiddlewareTest::testRejectCanonicalName
Failed asserting that exception message 'Host "192.168.0.2" (www.example.com:80) violates local access rules' contains 'Host violates local access rules'.

[Commifreak]

Uhm... Thanks for bringing that up. Iam not well experienced with testst and it feels wrong to hard-code the to-test IPs to the expectExceptionMessage function. Is there a way to define that the expected exception message contains not the exact string but the part violates local access rules?

EDIT: Sorry, just read the PHPUnit docs right, that IS actually a part of a string... I will update the test to only contain the last part.

[welcome]

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22