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.

Sign up for GitHub

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

Jump to bottom

Feature Request #81

Closed
xeor opened this issue Apr 7, 2024 · 1 comment · Fixed by #82
Closed

Feature Request #81

xeor opened this issue Apr 7, 2024 · 1 comment · Fixed by #82
Assignees
@Leo310 @nicobrauchtgit
Labels
documentation Improvements or additions to documentation

Comments

@xeor
Copy link

xeor commented Apr 7, 2024

Feature Area

Settings

Painpoint

Setting OLLAMA_ORIGIN to * hurts.. Feels more insecure than what it should be

Describe your idea

Document to set it to OLLAMA_ORIGINS='app://obsidian.md*' instead. It is a comma separated list and should contain more than just a star

Alternatives

Just leave it as documented, but maybe add some comment?

Additional Context

No response
@xeor xeor added the enhancement New feature or request label Apr 7, 2024
@nicobrauchtgit nicobrauchtgit added documentation Improvements or additions to documentation and removed enhancement New feature or request labels Apr 7, 2024
@nicobrauchtgit
Copy link
Collaborator

Yeah that is a fair point!

@nicobrauchtgit nicobrauchtgit mentioned this issue Apr 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Leo310 Leo310

@nicobrauchtgit nicobrauchtgit

Labels
documentation Improvements or additions to documentation
Projects
Dev planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update Documentation: OLLAMA_ORIGINS='app://obsidian.md' your-papa/obsidian-Smart2Brain
3 participants
@xeor @Leo310 @nicobrauchtgit