Skip to content

Pablo/refresh ip vpn#182

Merged
dapplion merged 6 commits intov0.2.8from
pablo/refreshIP-VPN
Dec 15, 2020
Merged

Pablo/refresh ip vpn#182
dapplion merged 6 commits intov0.2.8from
pablo/refreshIP-VPN

Conversation

@pablomendezroyo
Copy link
Contributor

@pablomendezroyo pablomendezroyo commented Dec 11, 2020 •
edited by dapplion
Loading

CHANGES:

  1. Change variable name in ovpn_get_client.sh file: OVPN_INTERNAL_IP --> DAPPNODE_INTERNAL_IP
  2. pollDappNodeConfig() refreshes env variable for internal IP.
  3. getClient() sets DAPPNODE_INTERNAL_IP from the refreshed env variable internalIP

Fixes #178

@pablomendezroyo
fetch internalIP from dappmanager in getClient()
ad926a6
@pablomendezroyo
Change default name for OVPN environment variable
a24a82f 
OVPN_INTERNAL_IP -->  DAPPNODE_INTERNAL_IP
@pablomendezroyo
Internal Ip refreshed
984a1e9 
PollDappNodeConfig() refreshes internalIP env variable. getClient() set the refreshed internalIP from the env variable into the OVPN file.
@dapplion dapplion changed the base branch from master to v0.2.8 December 11, 2020 08:46
dapplion
dapplion reviewed Dec 11, 2020
View reviewed changes
@pablomendezroyo pablomendezroyo linked an issue Dec 11, 2020 that may be closed by this pull request
VPN should fetch localIP every new VPN credentials created #178
Closed
@dapplion dapplion merged commit c958209 into v0.2.8 Dec 15, 2020
@dapplion dapplion deleted the pablo/refreshIP-VPN branch December 15, 2020 07:49
@dapplion dapplion mentioned this pull request Dec 15, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dapplion dapplion dapplion left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

VPN should fetch localIP every new VPN credentials created

2 participants

@pablomendezroyo @dapplion