Skip to content
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

Fix a typo in _index.md #6491

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Conversation

trungnguyen
Copy link
Contributor

missing 's' in the word 'public_server' in the code below data.example.public_server[s]

Why the changes in this PR are needed?

What are the changes in this PR?

Notes to assist PR review:

Further comments:

Copy link

netlify bot commented Dec 18, 2023

Deploy Preview for openpolicyagent ready!

Name Link
🔨 Latest commit f374f69
🔍 Latest deploy log https://app.netlify.com/sites/openpolicyagent/deploys/6595eac3a69e7600085986ae
😎 Deploy Preview https://deploy-preview-6491--openpolicyagent.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for helping make the docs better! I think we'll need to do more here though 😅

@@ -1002,7 +1002,7 @@ opa run example.rego repl.input:input.json
```

```ruby
> data.example.public_server[s]
> data.example.public_servers[s]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rule is defined as public_server here, though: https://github.com/open-policy-agent/opa/pull/6491/files#diff-5d9ba8bc1a568741b44ccc852748ca739d71f89dc1ac05a9f8d3787a064a549dR907

So either we keep it as-is, or change both locations (and perhaps others, if there are any).

@ashutosh-narkar
Copy link
Member

@trungnguyen thanks for working on this. If you can address Stephan's comments above that would be great. Thanks!

@@ -904,7 +904,7 @@ violation contains server.id if { # a server is in the violati
server.protocols[_] == "telnet" # it contains the "telnet" protocol.
}

public_server contains server if { # a server exists in the public_server set if...
public_servers contains server if { # a server exists in the public_servers set if...
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is called earlier in the policy. So we would need to update that too.

@ashutosh-narkar
Copy link
Member

Thanks for the changes @trungnguyen! Can you please squash your commits and sign them off and we can get this in. See this reference for authoring a good commit message.

@trungnguyen trungnguyen force-pushed the patch-1 branch 2 times, most recently from 65cf436 to c49af54 Compare January 3, 2024 23:33
@ashutosh-narkar ashutosh-narkar merged commit ee6799a into open-policy-agent:main Jan 4, 2024
24 checks passed
@trungnguyen trungnguyen deleted the patch-1 branch January 5, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants