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

feat: improve browser service #653

Merged
merged 6 commits into from
Nov 28, 2024
Merged

Conversation

cygaar
Copy link
Collaborator

@cygaar cygaar commented Nov 28, 2024

Relates to:

Performance and security improvements with the browser service.

Risks

Low since the browser performance was already inconsistent

Background

What does this PR do?

There are several changes I made to the browser service to make it better:

  • More accurate and up to date user agent to reduce bot detection
  • Remove redundant content summarization
  • Remove queuing system since all fetches are blocking and it was leading to duplicate pages being loaded
  • Run in headless mode to reduce cpu usage and latency
  • Re-enable sandbox mode to isolate malicious browser behavior
  • Prevent unwanted downloads of files

What kind of change is this?

Improvements

Documentation changes needed?

None

Testing

Where should a reviewer start?

Test by sending urls on discord

Detailed testing steps

@lalalune
Copy link
Member

v smort

@lalalune lalalune merged commit dbec9fd into elizaOS:main Nov 28, 2024
4 checks passed
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.

2 participants