-
Notifications
You must be signed in to change notification settings - Fork 16
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
Get geolocation about a given IP address #138
base: main
Are you sure you want to change the base?
Changes from 19 commits
28779ba
23df1d1
fc08d73
0b966f4
7224f68
e70054b
97c4ec6
f7246fc
5a3506c
37271a3
4473889
5a97097
77ca319
6f4f8be
84bd996
3cd7cf2
1dc6920
5ec7b1b
9616015
2a83ae9
9c5177e
629c057
665dd7d
44347b6
d9f1270
f284d67
262f214
9d67bc5
bbafc46
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
name: Deploy Compute@Edge GeoIp | ||
on: | ||
push: | ||
# branches: [main] | ||
## Note: by removing the branches value, the workflow will run when a push is made to any branch. This is to serve for testing. | ||
|
||
jobs: | ||
deploy: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
|
||
- name: Install project dependencies | ||
run: npm install | ||
working-directory: fastly-compute/geoip | ||
|
||
- name: Deploy to Compute@Edge | ||
uses: fastly/compute-actions@v5 | ||
with: | ||
project_directory: ./fastly-compute/geoip | ||
env: | ||
FASTLY_API_TOKEN: ${{ secrets.FASTLY_API_TOKEN }} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
/node_modules | ||
/bin | ||
/pkg |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
# Empty Starter Kit for JavaScript | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Update this doc with some helpful information! I'd suggest at a minimum:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Definitely makes sense, I'll be sure to work this read.me over. |
||
|
||
[![Deploy to Fastly](https://deploy.edgecompute.app/button)](https://deploy.edgecompute.app/deploy) | ||
|
||
An empty application template for the Fastly Compute@Edge environment which simply returns a 200 OK response. | ||
|
||
**For more details about other starter kits for Compute@Edge, see the [Fastly developer hub](https://developer.fastly.com/solutions/starters)** | ||
|
||
## Security issues | ||
|
||
Please see our [SECURITY.md](SECURITY.md) for guidance on reporting security-related issues. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
# This file describes a Fastly Compute@Edge package. To learn more visit: | ||
# https://developer.fastly.com/reference/fastly-toml/ | ||
|
||
authors = ["[email protected]"] | ||
description = "Test" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. suggestion: update description |
||
language = "javascript" | ||
manifest_version = 3 | ||
name = "geoip" | ||
service_id = "CnsNqIdpSj1tZNLTl6PUi4" | ||
|
||
[scripts] | ||
build = "npm run build" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We likely want to restore this behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed 👍