Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

Make use of cjdns for Prometheus/API/Gateway authentication #40

Merged
4 commits merged into from
Jul 6, 2015
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jul 6, 2015

  • gateway at http://[fc12::3456]:8080
  • api at http://[fc12::3456]:5001
  • prometheus at http://[fc12::3456]:80 / http://metrics.ipfs.io

All of them require authentication with a whitelisted
cjdns IPv6 address (see secrets.yml).

The other three commits are semi-random fixes.

Lars Gierth added 4 commits July 7, 2015 00:21
Ansible fails if the host is out of diskspace.

License: MIT
Signed-off-by: Lars Gierth <larsg@systemli.org>
License: MIT
Signed-off-by: Lars Gierth <larsg@systemli.org>
- gateway at [fc12::3456]:8080
- api at [fc12::3456]:5001
- prometheus at [fc12::3456]:80 / metrics.ipfs.io

All of them require authentication with a whitelisted
cjdns IPv6 address (see secrets.yml).

License: MIT
Signed-off-by: Lars Gierth <larsg@systemli.org>
License: MIT
Signed-off-by: Lars Gierth <larsg@systemli.org>
@ghost ghost added the solarnet label Jul 6, 2015
@ghost ghost mentioned this pull request Jul 6, 2015
@jbenet
Copy link
Member

jbenet commented Jul 6, 2015

I think this LGTM.

ghost pushed a commit that referenced this pull request Jul 6, 2015
Make use of cjdns for Prometheus/API/Gateway authentication
@ghost ghost merged commit 7b729fc into master Jul 6, 2015
@ghost ghost deleted the prom-auth branch July 6, 2015 22:57
@jbenet jbenet removed the in progress label Jul 6, 2015
@jbenet
Copy link
Member

jbenet commented Jul 6, 2015

prometheus at http://[fc12::3456]:80 does it have to be 80 ? maybe we could keep 80 allocated (for nginx)? (would be nicer to reuse the same ports) -- (i may be misunderstanding)

@ghost
Copy link
Author

ghost commented Jul 6, 2015

That's simply a vhost, which only gets hit if the request's Host header is metrics.ipfs.io -- anything else will hit the gateway

@jbenet
Copy link
Member

jbenet commented Jul 6, 2015

@lgierth ah ok sounds good 👍

@jbenet jbenet mentioned this pull request Jul 8, 2015
37 tasks
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant