YunoHost-Apps / vpnclient_ynh

VPN Client app for YunoHost
GNU Affero General Public License v3.0
41 stars 25 forks source link

A fix for the fix in PR #37 #40

Closed pitchum closed 6 years ago

pitchum commented 6 years ago

The problem

The fix proposed in PR #37 is broken. date -s "${http_date}" won't work if ${http_date} is expressed in seconds.

The solution

store http_date in two separate variables.

agentcobra commented 6 years ago

linked with #37 ?

pitchum commented 6 years ago

absolutely!

The fix in PR #37 is broken. Here is the fix for the fix.

alexAubin commented 6 years ago

ping @zamentur : does that sounds good to you ? :stuck_out_tongue:

zamentur commented 6 years ago

Seems good :)