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

[pushover] Feature request: Send an Image URL directly #9340

Closed
cweitkamp opened this issue Apr 23, 2018 · 0 comments · Fixed by #11027
Closed

[pushover] Feature request: Send an Image URL directly #9340

cweitkamp opened this issue Apr 23, 2018 · 0 comments · Fixed by #11027
Assignees
Labels
enhancement An enhancement or new feature for an existing add-on PR pending There is a pull request for resolving the issue

Comments

@cweitkamp
Copy link
Contributor

  • Sending a PNG image from internet: sendPushoverImage("Test", "https://jaxenter.de/wp-content/uploads/2017/12/openhablogo.png")

Doesn't work. But would be a very nice feature. Can you please add it. Thanks.

2018-03-29 12:18:30.558 [ERROR] [ab.action.pushover.internal.Pushover] - An error occurred while notifying your mobile device.
java.io.FileNotFoundException: File is not a normal file.
	at org.apache.commons.httpclient.methods.multipart.FilePartSource.<init>(FilePartSource.java:69) ~[?:?]
	at org.apache.commons.httpclient.methods.multipart.FilePartSource.<init>(FilePartSource.java:89) ~[?:?]
	at org.apache.commons.httpclient.methods.multipart.FilePart.<init>(FilePart.java:160) ~[?:?]
	at org.openhab.action.pushover.internal.Pushover.pushover(Pushover.java:368) ~[?:?]
	at org.openhab.action.pushover.internal.Pushover.sendPushoverImage(Pushover.java:107) ~[?:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:?]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
	at java.lang.reflect.Method.invoke(Method.java:498) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.invokeOperation(XbaseInterpreter.java:1085) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.invokeOperation(XbaseInterpreter.java:1060) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter._invokeFeature(XbaseInterpreter.java:1046) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.invokeFeature(XbaseInterpreter.java:991) ~[?:?]
	at org.eclipse.smarthome.model.script.interpreter.ScriptInterpreter.invokeFeature(ScriptInterpreter.java:143) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter._doEvaluate(XbaseInterpreter.java:901) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter._doEvaluate(XbaseInterpreter.java:864) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.doEvaluate(XbaseInterpreter.java:223) ~[?:?]
	at org.eclipse.smarthome.model.script.interpreter.ScriptInterpreter.doEvaluate(ScriptInterpreter.java:219) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.internalEvaluate(XbaseInterpreter.java:203) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter._doEvaluate(XbaseInterpreter.java:446) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.doEvaluate(XbaseInterpreter.java:227) ~[?:?]
	at org.eclipse.smarthome.model.script.interpreter.ScriptInterpreter.doEvaluate(ScriptInterpreter.java:219) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.internalEvaluate(XbaseInterpreter.java:203) ~[?:?]
	at org.eclipse.xtext.xbase.interpreter.impl.XbaseInterpreter.evaluate(XbaseInterpreter.java:189) ~[?:?]
	at org.eclipse.smarthome.model.script.runtime.internal.engine.ScriptImpl.execute(ScriptImpl.java:82) ~[?:?]
	at org.eclipse.smarthome.model.rule.runtime.internal.engine.RuleEngineImpl.lambda$2(RuleEngineImpl.java:339) ~[?:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:?]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:?]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180) [?:?]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293) [?:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:?]
	at java.lang.Thread.run(Thread.java:748) [?:?]

Example for current API: sendPushoverMessage(pushoverBuilder("Test").withAttachment("https://jaxenter.de/wp-content/uploads/2017/12/openhablogo.png"))

See openhab/openhab1-addons#5527 (comment) and https://community.openhab.org/t/pushover-image-support/39201/53.

@cweitkamp cweitkamp changed the title [Pushover] Feature request: Send an Image URL directly [pushover] Feature request: Send an Image URL directly Dec 12, 2020
@cweitkamp cweitkamp self-assigned this Dec 12, 2020
@cweitkamp cweitkamp transferred this issue from openhab/openhab1-addons Dec 12, 2020
@cweitkamp cweitkamp added the enhancement An enhancement or new feature for an existing add-on label Jan 12, 2021
@cweitkamp cweitkamp added the PR pending There is a pull request for resolving the issue label Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on PR pending There is a pull request for resolving the issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant