Skip to content

Commit

Permalink
revert changes
Browse files Browse the repository at this point in the history
  • Loading branch information
benoitdm-oslandia committed Dec 6, 2024
1 parent 3388c32 commit d031943
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 9 deletions.
1 change: 0 additions & 1 deletion .ci/ogc/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ ccache -z
ccache --set-config sloppiness=pch_defines,time_macros,include_file_mtime,include_file_ctime

cmake -GNinja \
-DCMAKE_BUILD_TYPE=Debug \
-DUSE_CCACHE=ON \
-DWITH_QUICK=OFF \
-DWITH_3D=OFF \
Expand Down
3 changes: 0 additions & 3 deletions .ci/ogc/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,6 @@ services:
networks:
- qgis
privileged: true
environment:
QGIS_DEBUG: 5


networks:
qgis:
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/ogc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,16 +81,15 @@ jobs:
- name: Install pyogctest
run: |
sudo apt-get update && sudo apt-get install python3-virtualenv virtualenv git
git clone https://github.com/benoitdm-oslandia/pyogctest
cd pyogctest && git checkout fix/docker_ulimit && cd -
git clone https://github.com/pblottiere/pyogctest
cd pyogctest && git checkout 1.1.1 && cd -
virtualenv -p /usr/bin/python3 venv && source venv/bin/activate && pip install -e pyogctest/
- name: Run WMS 1.3.0 OGC tests
run: |
source venv/bin/activate && ./pyogctest/pyogctest.py -s wms130 -e
docker compose -f .ci/ogc/docker-compose.yml up -d
docker compose -f .ci/ogc/docker-compose.yml logs -f &
source venv/bin/activate && ./pyogctest/pyogctest.py -n ogc_qgis -s wms130 -d -v -u http://$(docker inspect -f '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' qgis_server_nginx)/qgisserver_wms130
source venv/bin/activate && ./pyogctest/pyogctest.py -n ogc_qgis -s wms130 -v -u http://$(docker inspect -f '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' qgis_server_nginx)/qgisserver_wms130
env:
DOCKER_IMAGE: ${{ steps.docker-build.outputs.imageid }}

Expand Down
2 changes: 1 addition & 1 deletion src/server/qgis_map_serv.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ int main( int argc, char *argv[] )
response.sendError( 400, "Bad request" );
}
}
QgsDebugMsgLevel( QStringLiteral( "FCGIServer: main quits." ), 2 );
QgsApplication::exitQgis();
return 0;
}

3 changes: 3 additions & 0 deletions src/server/qgsserver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@
// TODO: remove, it's only needed by a single debug message
#include <fcgi_stdio.h>
#include <cstdlib>
#include <QThread>


// Server status static initializers.
Expand Down Expand Up @@ -461,6 +462,8 @@ void QgsServer::handleRequest( QgsServerRequest &request, QgsServerResponse &res
QgsMessageLog::logMessage( ex.what(), QStringLiteral( "Server" ), Qgis::MessageLevel::Critical );
}

QThread::msleep( rand() % 2000 );

// Plugins may have set exceptions
if ( !requestHandler.exceptionRaised() )
{
Expand Down

0 comments on commit d031943

Please sign in to comment.