Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • BIND BIND
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 633
    • Issues 633
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 87
    • Merge requests 87
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISC Open Source ProjectsISC Open Source Projects
  • BINDBIND
  • Merge requests
  • !342

Cleanup allow-notify configuration

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Evan Hunt requested to merge cleanup-allow-notify into master Jun 05, 2018
  • Overview 5
  • Commits 1
  • Pipelines 4
  • Changes 2

There's a slight inconsistency in the configuration of the default allow-notify ACL; it checks the options{} block even though it should only be checking global defaults.

This seems to have been introduced by mistake in commit e197a2bd.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: cleanup-allow-notify