mirror of
https://github.com/spatie/dnsrecords.io.git
synced 2026-04-27 08:25:57 +03:00
[GH-ISSUE #31] 403 blade view #15
Labels
No labels
bug
good first issue
help wanted
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/dnsrecords.io#15
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @willemvb on GitHub (Oct 31, 2017).
Original GitHub issue: https://github.com/spatie/dnsrecords.io/issues/31
Originally assigned to: @brendt on GitHub.
https://dnsrecords.io/...gives a 403.I've made a Blade template but it doesn't get picked up like other errors.
(BTW not sure why
https://dnsrecords.io/..kinda works andhttps://dnsrecords.io/...not)@freekmurze commented on GitHub (Oct 31, 2017):
Seems like nginx is tripping on
...This problem occurs on many different sites as well. Let's just filter out...client side for now.@brendt commented on GitHub (Nov 1, 2017):
@freekmurze I think a better approach would be to handle that error in the new middleware I added in #33, server side. Though
...might be the one exception which doesn't even reach Laravel's router.If it's possible, are you ok with filtering it out serverside?
@freekmurze commented on GitHub (Nov 1, 2017):
Merged #33
@brendt commented on GitHub (Nov 2, 2017):
@freekmurze unfortunately this URL never reaches Laravel's router:
dnsrecords.io/....One part of the solution is the filter
...uit client side, but the URL itself would still be shareable. I think the only way to really fix it, is to rewrite...in nginx.@freekmurze commented on GitHub (Nov 2, 2017):
Seems a problem across a lot of forge provisioned sites. I'm going to leave it as it is for now.