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

Sync to head #2034

Merged
merged 26 commits into from
Aug 30, 2022
Merged

Sync to head #2034

merged 26 commits into from
Aug 30, 2022

Conversation

scudette
Copy link
Contributor

No description provided.

scudette and others added 26 commits August 16, 2022 12:58
Make cookie and JWT expiry configurable
* Used by process_tracker_tree() to build a process tree
* Fixed linux pslist() which was very slow due to including a lot of
  unnecessary and expensive fields. We now only return the commonly
  used fields
* Collect domain role info on interrogate

If populated on check in, domainrole can be used to auto-tag or filter down for certain hunts (ei: Domain controllers)

Ref: https://docs.microsoft.com/en-us/windows/win32/cimwin32prov/win32-computersystem#:~:text=workgroup%20is%20returned.-,DomainRole,-Data%20type%3A

* Cleaned up domain role lookup and added a notebook suggestion

Co-authored-by: Mike Cohen <mike@velocidex.com>
…TH2 (#2000)

I was able to crash Velociraptor by requesting the github authenticator
callback URL directly with e.g. curl https://vrrserver/auth/github/callback

It turns out that there was no error handling if there is no 'oauthstate'
cookie provided as part of the request and we hit a nil pointer
dereference panic.  The Google and Azure authenticators had the same
issue.

This commit fixes all three and resolves #1999.
- IP field was not properly parsed - replaced with a parse_binary()
  version to ensure backwards compatibility.
- By default parse_ese() was using the "file" accessor which in 0.6.5
  was changed to not fallback to NTFS parsing. This means that since
  UAL files are locked, the parser was unable to access them. This PR
  sets the accessor to be "auto" explicitly thereby forcing the ntfs
  parsing if needed.
Added rolename mappings and updated details.
Also allow the query command to specify an org id.
This makes it more efficient and simpler to filter by large number of
regex without adding a lot of AND clauses to the query.
Now all regex must match all items.
When performing a full sync (e.g. pslist), some of the processes have
no valid parent at this time (because the parent e.g. exited).

We need to mark those unknown parents in case a new process reuses
those pids - in this case the process call chain can accidentally
include those parents.
* Refactor code to propagate the context in more cases.

* Fixed tests
* update to clean up null fields
* update to clean up null fields tests
* Add embedded stager parse usecase

* Add some test fixes

* Add test results

* Add test fix
This brings it in line with the same column name on Windows.

Also fixed a crash in user_grant() due to insufficient error
checking.

Fixes: #2022
When a user is created with no access to the root org, the GUI did not
automatically switch the user to their own org. This caused an issue
where the user was rejected (because by default they were trying to
access the root org) but there was no way to switch even manually to
the correct org.

This PR updates the user's preferences to the first available org
automatically allowing the user to log in and select other orgs
manually.
Also made maximum VFS directory size configurable.

Fixes: #2005
When following ETW the EventData is an unordered map so we need to
explicitly build a dict() to maintain consistent ordering.

Also fixed bug in USN artifact
This caused it to stop emitting rows after a while because it was
unable to see new data.
fix: upgrade ace-builds from 1.8.1 to 1.9.3

Snyk has created this PR to upgrade ace-builds from 1.8.1 to 1.9.3.

See this package in npm:
https://www.npmjs.com/package/ace-builds

See this project in Snyk:
https://app.snyk.io/org/scudette/project/76f4d127-566b-42ef-86f4-bdcbc92b90b4?utm_source=github&utm_medium=referral&page=upgrade-pr
@CLAassistant
Copy link

CLAassistant commented Aug 30, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
6 out of 7 committers have signed the CLA.

✅ scudette
✅ jeffmahoney
✅ svch0stz
✅ weslambert
✅ mgreen27
✅ baileys20055
❌ snyk-bot
You have signed the CLA already but the status is still pending? Let us recheck it.

@scudette scudette merged commit edc1369 into v0.6.6 Aug 30, 2022
@scudette scudette deleted the 0.6.6-merge branch August 30, 2022 14:06
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.

8 participants