Skip to content

Commit

Permalink
fix http headers in OWS + fix test
Browse files Browse the repository at this point in the history
  • Loading branch information
3nids committed Jan 16, 2023
1 parent 1029bce commit a88ec8b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/core/qgsowsconnection.cpp
Expand Up @@ -143,7 +143,7 @@ QgsDataSourceUri &QgsOwsConnection::addWmsWcsConnectionSettings( QgsDataSourceUr
{
addCommonConnectionSettings( uri, service, connName );

settingsHeaders->setValue( uri.httpHeaders().headers(), {service.toLower(), connName} );
uri.setHttpHeaders( QgsHttpHeaders( settingsHeaders->value( {service.toLower(), connName} ) ) );

if ( settingsIgnoreGetMapURI->value( {service.toLower(), connName} ) )
{
Expand Down
7 changes: 2 additions & 5 deletions tests/src/core/testqgshttpheaders.cpp
Expand Up @@ -143,11 +143,8 @@ void TestQgsHttpheaders::updateSettings()

void TestQgsHttpheaders::createQgsOwsConnection()
{
QgsSettings settings;
settings.setValue( QString( QgsSettings::Prefix::QGIS ) + "/connections-service/name/" + QgsHttpHeaders::PATH_PREFIX + QgsHttpHeaders::KEY_REFERER,
"http://test.com" );
settings.setValue( QString( QgsSettings::Prefix::QGIS ) + "/connections-service/name/" + QgsHttpHeaders::PATH_PREFIX + "other_http_header",
"value" );
QgsHttpHeaders h( QVariantMap( { { QgsHttpHeaders::KEY_REFERER, "http://test.com"}, {"other_http_header", "value"}} ) );
QgsOwsConnection::settingsHeaders->setValue( h.headers(), {"service", "name"} );

QgsOwsConnection ows( "service", "name" );
QCOMPARE( ows.connectionInfo(), ",authcfg=,referer=http://test.com" );
Expand Down

0 comments on commit a88ec8b

Please sign in to comment.