Skip to content

Conversation

Akasch
Copy link
Member

@Akasch Akasch commented Feb 9, 2020

This adds a module to keep track of slow to generate tiles
The tileserver can get the list and pre generate them

Missing features

  • Upload the generated tiles
  • auth of the tileserver
  • request with since flag

@Akasch Akasch requested a review from lilioid February 9, 2020 21:26
@Akasch Akasch self-assigned this Feb 9, 2020
This adds a module to keep trak of slow to generate tiles
The tileserver can get the list and pregenerate them
@Akasch Akasch force-pushed the feature_slowtiles branch from 2a37164 to 2ec2821 Compare February 9, 2020 21:54
@Akasch
Copy link
Member Author

Akasch commented Feb 9, 2020

For reference the command to parse the nginx logfile into the database:

cat /var/log/nginx/master.tiles.map-data.de.error.log | grep 'timed out' | sed 's|.*://.*/all/\([^?]*\)\.mvt.*|\1|g' | while read line || [[ -n $line ]] ; do curl "http://127.0.0.1:<port>/slow_tiles/add/${line}"; done

Copy link
Contributor

@lilioid lilioid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are django apps proper python modules? Meaning don't you need a slowtiles/__init__.py file?

@require_POST
@csrf_exempt
def upload_tile(request, z, x, y, format):
# TODO auth oth server
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is auth oth server?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be authenticate the server, because at the moment all servers having access to the api can upload files

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this can be fixed by using #8 and adding the checks here to

@Akasch Akasch mentioned this pull request Feb 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants