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

Rule name display is not complete #121

Closed
daikerSec opened this issue May 14, 2020 · 5 comments
Closed

Rule name display is not complete #121

daikerSec opened this issue May 14, 2020 · 5 comments
Labels
information_needed needs-testing Fix needs testing stale No activity - Closed

Comments

@daikerSec
Copy link

Rule name display is not complete when ruler display. For example, I have a Rule Name called AllStaff, which will be displayed as llStaff
ruler

@staaldraad
Copy link
Collaborator

Hi

Is this with the latest version? There was a bug in rule parsing before, this might be a regression. Otherwise it looks like the first rule isn't printing correctly either, that might have something to do with it.
Could you please provide the version of Ruler you are using, and whether this is MAPI/HTTP or RPC/HTTP.

@daikerSec
Copy link
Author

daikerSec commented May 25, 2020

Hello, I have encountered this problem in both 2.2.2 and 2.4.0.

image
image

This is the command I used

ruler_windows_amd64.exe --insecure --url https://autodiscover.xx.xx.xxx/autodiscover/autodiscover.xml  --email [email protected] -u xxx -p xxxx --verbose --debug display

@daikerSec
Copy link
Author

Hello, I have encountered this problem in both 2.2.2 and 2.4.0.

image
image

This is the command I used

ruler_windows_amd64.exe --insecure --url https://autodiscover.xx.xx.xxx/autodiscover/autodiscover.xml  --email [email protected] -u xxx -p xxxx --verbose --debug display

@staaldraad

@staaldraad
Copy link
Collaborator

Hey @daikerSec

This seems like it might be an issue with unicode. Do the Rule IDs still print correctly and are usable directly? I've got no way to test this unfortunately. It might also be worth trying in a Linux terminal or another terminal emulator for Windows, just to rule out that it isn't an output problem

@staaldraad
Copy link
Collaborator

I think this might be due to the bug pointed out in #122

I've drafted a release that fixes it, if you could give it a try please @daikerSec : https://github.com/sensepost/ruler/releases/tag/untagged-ec15daae6305be4005f2

@staaldraad staaldraad added needs-testing Fix needs testing stale No activity - Closed labels May 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
information_needed needs-testing Fix needs testing stale No activity - Closed
Projects
None yet
Development

No branches or pull requests

2 participants