Skip to content
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

Printing completely broken in 1.2b1 #340

Closed
jj0hns0n opened this issue Jul 27, 2012 · 9 comments
Closed

Printing completely broken in 1.2b1 #340

jj0hns0n opened this issue Jul 27, 2012 · 9 comments
Assignees
Milestone

Comments

@jj0hns0n
Copy link
Member

from @cholmes "Printing didn't work at all. Always got The server returned an error: 500 Internal Server Error (tried a few times on different types of layers)"

@cholmes
Copy link

cholmes commented Jul 27, 2012

I don't think this is critical for 1.2. But if it's not working at all we should probably remove that gxp button from map composer.

@jj0hns0n
Copy link
Member Author

I think this may still have to do with the MapFish whitelist etc. I will
check into it.

On Fri, Jul 27, 2012 at 12:44 PM, Chris Holmes <
reply@reply.github.com

wrote:

I don't think this is critical for 1.2. But if it's not working at all we
should probably remove that gxp button from map composer.


Reply to this email directly or view it on GitHub:
#340 (comment)

@ghost ghost assigned jj0hns0n Jul 27, 2012
@ingenieroariel
Copy link
Member

This is documented in the post-install notes [1] so we can leave it as is. I think this is a fix for the geoserver printing module, to at least print from the local the geoserver without adding it to the whitelist.

[1] http://docs.geonode.org/en/latest/deploy/production.html#configure-the-printing-module

@jj0hns0n
Copy link
Member Author

I seem to be running into problems with the normal set of base layers we
have, even after adding the entry for the local one. I think we need to
make sure that any thing we have set in the base layers by default is
whitelisted in this yaml.

That said, we are moving toward replacing this mapfish module with
something more 'standards based' in the near future, so I think it may be
ok to just disable the print button and leave this for 2.x.

On Fri, Jul 27, 2012 at 7:05 PM, Ariel Núñez <
reply@reply.github.com

wrote:

This is documented in the post-install notes [1] so we can leave it as is.
I think this is a fix for the geoserver printing module, to at least print
from the local the geoserver without adding it to the whitelist.

[1]
http://docs.geonode.org/en/latest/deploy/production.html#configure-the-printing-module


Reply to this email directly or view it on GitHub:
#340 (comment)

@ingenieroariel
Copy link
Member

Re-enabling the print button is harder than just fixing the printing
template. (Seems like it would require fetching the js sources,
recompiling them and putting them in the right place).

I am -1 on disabling the print button. And +1 on adding entries for
the default base layers, we have this as one of the required steps
after installation (along with creating a superuser and setting
SITEURL) [1].

[1] http://docs.geonode.org/en/latest/deploy/production.html#required-steps

On Fri, Jul 27, 2012 at 11:14 PM, Jeffrey Johnson
reply@reply.github.com
wrote:

I seem to be running into problems with the normal set of base layers we
have, even after adding the entry for the local one. I think we need to
make sure that any thing we have set in the base layers by default is
whitelisted in this yaml.

That said, we are moving toward replacing this mapfish module with
something more 'standards based' in the near future, so I think it may be
ok to just disable the print button and leave this for 2.x.

On Fri, Jul 27, 2012 at 7:05 PM, Ariel Núñez <
reply@reply.github.com

wrote:

This is documented in the post-install notes [1] so we can leave it as is.
I think this is a fix for the geoserver printing module, to at least print
from the local the geoserver without adding it to the whitelist.

[1]
http://docs.geonode.org/en/latest/deploy/production.html#configure-the-printing-module


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)

@jj0hns0n
Copy link
Member Author

I still Ran into problems even after adding all the base layers. I think
it's not respecting sub domains porperly. @ingenieroariel if you can get it
to work by modifying the yaml, I'm happy to leave it in, if not, no sense
including a broken feature.

On Saturday, July 28, 2012, Ariel Núñez wrote:

Re-enabling the print button is harder than just fixing the printing
template. (Seems like it would require fetching the js sources,
recompiling them and putting them in the right place).

I am -1 on disabling the print button. And +1 on adding entries for
the default base layers, we have this as one of the required steps
after installation (along with creating a superuser and setting
SITEURL) [1].

[1]
http://docs.geonode.org/en/latest/deploy/production.html#required-steps

On Fri, Jul 27, 2012 at 11:14 PM, Jeffrey Johnson
<reply@reply.github.com javascript:;>
wrote:

I seem to be running into problems with the normal set of base layers we
have, even after adding the entry for the local one. I think we need to
make sure that any thing we have set in the base layers by default is
whitelisted in this yaml.

That said, we are moving toward replacing this mapfish module with
something more 'standards based' in the near future, so I think it may be
ok to just disable the print button and leave this for 2.x.

On Fri, Jul 27, 2012 at 7:05 PM, Ariel Núñez <
reply@reply.github.com javascript:;

wrote:

This is documented in the post-install notes [1] so we can leave it as
is.
I think this is a fix for the geoserver printing module, to at least
print
from the local the geoserver without adding it to the whitelist.

[1]

http://docs.geonode.org/en/latest/deploy/production.html#configure-the-printing-module


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)

@ingenieroariel
Copy link
Member

Jeff can you share the error you get and the modifications to made to the yaml?

-a

On Sat, Jul 28, 2012 at 4:28 PM, Jeffrey Johnson
reply@reply.github.com
wrote:

I still Ran into problems even after adding all the base layers. I think
it's not respecting sub domains porperly. @ingenieroariel if you can get it
to work by modifying the yaml, I'm happy to leave it in, if not, no sense
including a broken feature.

On Saturday, July 28, 2012, Ariel Núñez wrote:

Re-enabling the print button is harder than just fixing the printing
template. (Seems like it would require fetching the js sources,
recompiling them and putting them in the right place).

I am -1 on disabling the print button. And +1 on adding entries for
the default base layers, we have this as one of the required steps
after installation (along with creating a superuser and setting
SITEURL) [1].

[1]
http://docs.geonode.org/en/latest/deploy/production.html#required-steps

On Fri, Jul 27, 2012 at 11:14 PM, Jeffrey Johnson
<reply@reply.github.com javascript:;>
wrote:

I seem to be running into problems with the normal set of base layers we
have, even after adding the entry for the local one. I think we need to
make sure that any thing we have set in the base layers by default is
whitelisted in this yaml.

That said, we are moving toward replacing this mapfish module with
something more 'standards based' in the near future, so I think it may be
ok to just disable the print button and leave this for 2.x.

On Fri, Jul 27, 2012 at 7:05 PM, Ariel Núñez <
reply@reply.github.com javascript:;

wrote:

This is documented in the post-install notes [1] so we can leave it as
is.
I think this is a fix for the geoserver printing module, to at least
print
from the local the geoserver without adding it to the whitelist.

[1]

http://docs.geonode.org/en/latest/deploy/production.html#configure-the-printing-module


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)


Reply to this email directly or view it on GitHub:
#340 (comment)

@jj0hns0n jj0hns0n mentioned this issue Jul 29, 2012
@jj0hns0n
Copy link
Member Author

Copying in info from #329 which was closed as a duplicate.

How can i sent geonode to print using both IP address . The following address is beiong displayed when I try to print a map

Error while generating PDF: DocumentException: org.mapfish.print.InvalidJsonValueException: spec.layers[0].baseURL has an invalid value: http://tile.openstreetmap.org/ (http://192.168.0.116/) at org.mapfish.print.map.readers.HTTPMapReader.checkSecurity(HTTPMapReader.java:64) at org.mapfish.print.map.readers.HTTPMapReader.(HTTPMapReader.java:55) at org.mapfish.print.map.readers.TileableMapReader.(TileableMapReader.java:40) at org.mapfish.print.map.readers.OsmMapReader.(OsmMapReader.java:41) at org.mapfish.print.map.readers.OsmMapReader.create(OsmMapReader.java:78) at org.mapfish.print.map.readers.MapReader.create(MapReader.java:56) at org.mapfish.print.map.MapChunkDrawer.renderImpl(MapChunkDrawer.java:98) at org.mapfish.print.ChunkDrawer.render(ChunkDrawer.java:55) at

@ingenieroariel
Copy link
Member

I had no trouble with this on a snapshot from current master after setting the local ip and adding.

In order to be able to fix this ticket I did three things:

  • Implemented geonode-updateip, an script that takes in an ip address or dns and updates SITEURL in local_settings.py and printing/config.yaml adding the configured ip address. (Already in master)
  • Moved printing config out of gs-data directory and into the data dir (previous attempt to add open layers and others by default had failed because config.yaml was not in the right place and a default version was then written by geoserver
  • Added maps.opengeo.org, OSM, Mapquest imagery and Mapquest OSM (the default layer) to the whitelist, so people have better chance of not running into trouble.

Ariel.

afabiani pushed a commit that referenced this issue Jun 5, 2018
afabiani pushed a commit that referenced this issue Oct 18, 2018
* Remove an ipdb line

* Enable DB_DATASTORE when using WorldMap

* Now it is possible to edit and style also from local dev

* Fixes #263

* Fixes #273

* Synced with geonode master

* Fixes a problem with layers upload

* Add url dispatcher for worldmap api

* WorldMap api version is now 2.8

* Added worldmap geoexplorer client source code. Fixes #265

* Fixed javascript build.xml and a few things in GeoExplorer.js

* Ported the action model and its api, which is needed by hypermap

* Updating instructions for geonode-worldmap

* Fixes #264

* Fixes #280

* Update instructions adding the configuration of update_last_wm_geonode_layers

* Trying to increase accesstoken expiration to see if this affects #283

* Removed google earth tool. Fixed the gxp_mapshare tool. Fixing google map api key read

* Adding a missing import which caused an error when looking at map page as anonymous user

* Migrate worldmap map revisions. Fixes #266

* Restore map print tool

* Fixes #267

* Fixes #260 (gazetteer)

* Forgot one requirement in previous commit

* Fixes #272

* Fixes permission issues with editing and use the correct source for wm layers

* Fixes #283

* Fixes #314

* Fixes #298

* Point to the correct warper site in the rectify images dialog

* Correctly identify a local layer when using GeoNodeQueryTool

* Fixes #322

* Fixes zoom to extent tool

* Fixes the regression casuing all the extent issues

* Fixes #301

* Release 2.8.0

* Updated changelog for version 2.8

* Update README

* Update README

* Add the feature search functionality

* Forgot file in previous commit

* Improve map thumbnails

* Make layer configuration in json map more robust

* Removed a stale file

* Adds a status message when updating the gazetteer fields for a layer

* Fixes #334

* Fixes #300

* Fixes #330

* Fixes #327

* Fixes #336

* Fixes #340

* Align to master branch

* Fixes #343

* Prepare 2.8.1

*  - Fix issue with layer upload

*  - Fix issue with layer upload

* Backport master fixes

*  - Fix reproj issue on bbox_to_projection

* fix slow login/logout on certain circumstances

It seems connexion was make through public location,
while it's an internal call, making the request
very slow on some setups (not sure geoserver login
works anyway...)

* use geoip2 for monitoring

* Use HTML widget in GXP for any field starting with "descriptio". Refs #348

* Constrain pip to 9.0.3

* Updated changelog for version 2.8

* Backporting Master PRs

* Fixes #350

* Re-enable thumbnails for layers. Fixes #351

* Now categories order is respected for existing maps. Refs #341

* Sync with GeoNode 2.8 part 1/2

* Updating geoexplorer to last version

* Fixes a couple of things which were broken by merge with 2.8.0

* Fixing a couple of things broke when merging

* renaming celery to celery_app

* Fixes the updatelayers command

* Update helpers.py

* Backporting Master PRs

* Backporting Master PRs

* Fixes #367

* Fixes 31

Adding the {% trans %} tag for translation.

* Fixes https://github.com/camp-zju/geonode/issues/32

Add lable _() for translation

* Fixes https://github.com/camp-zju/geonode/issues/33

Add "Text" in .js file for translation

* Fixes https://github.com/camp-zju/geonode/issues/34

add Tianditu basemap

* Add Chinese translation file

* Add Chinese translation file

* Fixes https://github.com/camp-zju/geonode/issues/36

* Modify the file for translation

* Fixes https://github.com/camp-zju/geonode/issues/37

* Some improvement to the createlayer application

* Fixes part of #358 (the layer extent)

* Now thumbanils are not generate from layers which are created. Fixes #358

* Backporting Master PRs

* Fixes #3801

* Fixes #375

* Backporting Master PRs

* Update integration.py

*  - Backport fixes from master

*  - Backport fixes from master

*  - Allow registered users to edit Remote Services

* Updating the client

* Move the worldmap documentation to the correct place

* Removed stale worldmap documentation page

*  - Backport commit 15123a5 from master

* A couple of fixes and removing geoexplorer source code

*  - Backport commit 6c0e8ca from master

* Removing from git the compiled geoexplorer worldmap client

* Removing worldmap account, which will be part of the cga geonode project

* Removing all static files, which should be added by pip install worldmap-geoexplorer

* Fixing a number of things before sending PR to GeoNode 2.7.x

* Updated requirements for WorldMap

* Including a local_settings sample file for worldmap

* Now using django-geoexplorer-worldmap from the pypi package

*  - Backport fixes from master branch

* Updated worldmap installation documentation

* Removed the worldmap.queue application for now

* Reset a couple of files

*  - Backport from master

* Fixes pep8 violations

* By default USE_WORLDMAP is False

* Fixes PEP8 violations and a syntax error

* Fixes a problem with createlayer app

* By default we dont use hypermap

* [Backport fixes from master]

* [Backport fixes from master]

* Backport from master branch

*  - SITEURL rstrip (/) consistently

* [Fixes #3800] Uploading shapefiles without a datefield and time-enabled is False in importer settings fails in 2.7.x

*  - Backport stable fixes from master branch

*  - Backport stable fixes from master branch

*  - Backporting Docker Improvs and Fixes from master branch

* Update Dockerfile

* Externalize OGC TIMEOUT setting as ENV var

*  - Fix celery initialization when using GeoNode ad a depenency

* pep8 issues

* - Fix celery initialization when using GeoNode ad a depenency

* - Fix celery initialization when using GeoNode ad a depenency

* - Fix celery initialization when using GeoNode ad a depenency

* pep8 fixes

* - Fix celery initialization when using GeoNode ad a depenency

*  - Fixes layer replace

* Update helpers.py

* Update helpers.py

* add missing ast import

docker compose would fail due to a missing import, looking for global ast

* [backport 2.7.x] Minor improvements: allow registered users to invite others / improve French translation

*  - Packagin scripts updates

*  - Fix max zoom issue

*  - backport from master

*  - Backport fixes and PRs from master

*  - Minor Layout improvements

*  - Backport fixes and PRs from master (#3846)

*  - Backport fixes and PRs from master

*  - Minor Layout improvements

* Backport fixes from master branch

* Backport fix #3853

* Backport #3856

*  - Backporting master branch fixes

*  - Backporting fixes from master

* Backport fixes from master

* Backport fixes from master

*  - Fix integration tests

* Backport fixes from master

*  - Fix typo

* Ported the mobile client as per #404

* Includes commands in Makefile to create and remove databases needed when using the worldmap contrib application

* Now there is not a local_settings.py file, so we need to set DATABASES when using worldmap application

* Updated worldmap installation instructions and setup_worlmap make command

* When parsing ALLOWED_HOSTS the error can be more generic (in my case got a syntax error)

* Added the initial migration for datastore_shards app

* Fix issue with styles introduced with latest GeoNode synced code

* Correctly reference the google maps api

* add some localizations

* Skip a layer in the map if its source is missing

* Add localization

* Uncommenting a line wrongly commented out

* Started the GeoFence refactoring

* Fixes a problem with owner permissions not being synced on geofence

* Use psycopg2 for syncing geofence if there is a GEOFENCE_URL setting

* Handle a case of possible null values from geofence database

* Adding a GEOFENCE_URL in OGC_SERVER

* Fixes permissions sync with geofence for anonymous users

* Fixes an issue with guardian anonymous permissions

* Fixes another issue with anonymous user

* Skipping not anymore existing layers in maps

* Fixes the google maps api path

* Fixes #441

* Fixes #439

* Make sure coordinates are not flipped for a wm map

* Removing an unwanted print statment

* Fixes #454

* Fixes after sync with master

* Fixes pep8 violations

* Fixes some problem with automatic merge and migrations
marthamareal pushed a commit to marthamareal/geonode that referenced this issue Sep 24, 2021
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

No branches or pull requests

3 participants