Comments (4)
It works when not using quotes:
Script:
mail.sh -s "Backrest" -m {{ .ShellEscape .Summary }}
But that is not the usual way to use parameters, but ok as long as ".ShellEscape" works reliably...
from backrest.
Hey, .ShellEscape is actually adding single quotes around the value for you in addition to escaping the contents, this should be the expected behavior. I think the addition of the double quotes was causing variable substitution / subcommands / etc to evaluate.
from backrest.
Hi
Simply adding ' around is really not enough.
The output of .Summary etc. must be parsed and every ' in it must have a backslash in front. That must be done by backrest because there is no way to securely paste strings with '"` etc. as parameter.
Greetings
from backrest.
More specifically it's using https://pkg.go.dev/github.com/alessio/shellescape , one of the effects is the single quoting but it's expected to escape symbols, other quotes, etc. I think the problem was the double quotes around the shell escaped expression .
from backrest.
Related Issues (20)
- No more weekly schedule after 1.0.0 HOT 4
- Prune, check, and forget operations fail HOT 6
- Add the ability to filter the snapshots by tags HOT 2
- Issue starting Backrest in Kubernetes without specifying BACKREST_PORT HOT 2
- Cron is not honored HOT 2
- i18n (internationalization): support non-english translation sets for the Backrest UI
- Retention Policy keeps wrong daily backup? HOT 2
- Tray/status menu icon on MacOS HOT 1
- Restore to path should actually do what it says HOT 2
- Show repository version and offer to upgrade if v1 HOT 3
- Time zone issue for plans, checks, and prunes HOT 2
- command output is not valid JSON: invalid character 'L' looking for beginning of value HOT 6
- Restore failure + recovery leaves plan in a failed state HOT 6
- Gotify Hook: Gotify Base URL does not accept just a hostname: 0,actionGotify,baseUrl is not a valid url HOT 3
- Setting of backup flags not possible on mobile browsers HOT 4
- error: failed to backup HOT 7
- Support to stop and start other docker containers before and after backing up HOT 11
- Download without restore HOT 2
- Dashboard with big links & contextual data
- Doc improvement: using custom CA with docker installation. HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from backrest.