pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Julian Simioni	653cbf8770	Default to 500 error This is not production ready code.	6 years ago
Julian Simioni	328504f98a	Use Node.js 10 Docker baseimage	6 years ago
Julian Simioni	47f3f87e72	Fix README formatting	6 years ago
Julian Simioni	2dd2f88ea7	Merge pull request #1198 from pelias/service-configuration-docs feat(docs): Add examples of using service `timeout` and `retry` settings	6 years ago
Julian Simioni	192ffc13dd	feat(docs): Add examples of using service `timeout` and `retry` settings These examples have been possible from the beginning, but not explicitly called out in documentation. Fixes https://github.com/pelias/wof-admin-lookup/issues/192	6 years ago
Julian Simioni	8751f1ae65	Merge pull request #1197 from pelias/libpostal_bugfixes apply parser bugfix for incorrect parsing of diagonal directionals	6 years ago
Peter Johnson	4d9ee0053b	feat(libpostal): patch parser output when confused by directionals	6 years ago
Julian Simioni	bee6fb7f94	Merge pull request #1196 from pelias/test-running-script chore(tests): Update test running script with latest info	6 years ago
Julian Simioni	8f159ceda7	chore(tests): Update test running script with latest info Use npx instead of calling binaries in node_modules, and link to relevant issue. Connects https://github.com/pelias/pelias/issues/744	6 years ago
Julian Simioni	1f56ba1edb	Merge pull request #1195 from pelias/remove-old-packages Remove old packages	6 years ago
Julian Simioni	3ad9c342c5	feat(deps): remove nsp dev dependency The NSP service is shutting down, and as far as I know we have not used this tool frequently at all. NPM has better tools built in now, like `npm audit`.	6 years ago
Julian Simioni	e588b92c5e	feat(deps): Remove uglify devDependency As I recall, this was only added to mitigate issues with running `npm ls` and correctly showing all dependencies installed (see https://github.com/pelias/api/issues/1179).	6 years ago
Julian Simioni	1808390e83	Merge pull request #1191 from pelias/use-pelias-user feat(docker): Run API processes using pelias user	6 years ago
Julian Simioni	1753eb5a9c	feat(docker): Run API processes using pelias user This is enabled by the work in https://github.com/pelias/baseimage/pull/2 and lets us ensure the API is run as a non-root user. Additionally the maintainer label is now set in the baseimage as of (https://github.com/pelias/baseimage/pull/8), so it can be removed here.	6 years ago
Julian Simioni	66fb60c4d8	Merge pull request #1187 from pelias/semantic-release-build-stage Replace semantic-release dependency with TravisCI build stages	6 years ago
Julian Simioni	fe69061c0c	Update Docker version This fixes an issue with the version of Alpine we're using:	6 years ago
Julian Simioni	3360640c43	Only run semantic-release on production branch	6 years ago
Julian Simioni	b5ed140111	Use docker image build script from ci-tools repo	6 years ago
Julian Simioni	2de274954f	Use ci-tools script for semantic-release	6 years ago
Julian Simioni	34e3e5fd86	feat(release): replace semantic-release dep with Travis build stages This change moves semantic-release out of dev-dependencies, but keeps its functionality by calling semantic-release as usual in a `release` TravisCI build stage. There are several advantages to this method: 1.) semantic-release is run only after all builds succeed. Our previous approach could have theoretically run semantic-release when some Node.js versions failed with the current code 2.) semantic-release (and it's many dependencies) are removed from `node_modules`. This increases the speed of `npm install` in all cases, and reduces the size of our Docker images by 20MB (from 284MB to 264MB)! Since the only time semantic-release is needed is on TravisCI anyway, it seems pointless that every installation of Pelias should include it. 3.) Because semantic-release is not in `package.json`, Greenkeeper will not attempt to update it. Semantic release updates _very_ frequently, and each update attempt seems to have a decent chance of experiencing a random TravisCI failure, causing unwanted notifications. There are probably downsides to this approach. For example, we should consider pinning the major version of semantic release during install. Additionally, and for obvious reasons, we can't fully test this change until it's merged to the `production` branch. We should consider testing it first on a lower priority repository. If this change _does_ work well, we should consider adopting it everywhere.	6 years ago
Joxit	7d9e3e29fd	feat(findbyid): Add lang query param for placeholder This will reduce network transfer and speedup requests Related: https://github.com/pelias/placeholder/pull/128	6 years ago
Julian Simioni	5b76c0d649	Merge pull request #1190 from pelias/greenkeeper/pelias-logger-1.2.0 fix(package): update pelias-logger to version 1.2.0	6 years ago
greenkeeper[bot]	67a3ab72a9	fix(package): update pelias-logger to version 1.2.0 Closes #1189	6 years ago
Julian Simioni	fd003dc76e	Merge pull request #1185 from pelias/move-logging-to-debug Move lots of logging from info to debug	6 years ago
Julian Simioni	800eb8ca03	Move lots of logging from info to debug These are log lines that are not really useful in a production context, and just create a lot of noise.	6 years ago
Julian Simioni	16760795e9	feat: Merge pull request #1181 from worace/hw/iso-package-migration Migrate to internal iso3166 definitions	6 years ago
Horace Williams	ed0a96cff2	Migrate to internal iso3166 defnitions This is a somewhat roundabout fix to #1179, as a way to deal with the persistent npm ls and commit hook troubles we were dealing with due to dependencies of the iso3166 package. Additionally it should give us a faster definition of these ISO lookups, since the existing approaches were implemented using linear scans through an array rather than map-based lookups.	6 years ago
Julian Simioni	d466e02c1d	fix: Merge pull request #1174 from pelias/attribution-docs Add `attributionURL` parameter to configuration documentation	6 years ago
Julian Simioni	b9353ef64d	Add `attributionURL` parameter to configuration documentation This should have been added in #1149	6 years ago
Julian Simioni	4b43029ea3	feat: Merge pull request #1180 from worace/hw/cli-entry-point Add executable for pelias api NPM package	6 years ago
Horace Williams	8fea8ac5af	Use pelias-api as executable name	6 years ago
Horace Williams	d541361210	Add executable for pelias NPM package	6 years ago
Julian Simioni	c2ce1cab31	feat: Merge pull request #1178 from worace/hw/remove-quattro Remove Quattroshapes deprecation notice	6 years ago
Julian Simioni	58b98a4aa4	fix: Merge pull request #1175 from Joxit/placeholder-geometricFiltersApply Always apply geometric filter for placeholder	6 years ago
Horace Williams	c7ae37980d	Remove Quattroshapes deprecation notice Fixes #1132 The Quattroshapes source has been deprecated from some time now, and any users for whom the deprecation notice was helpful have presumably either upgraded or moved on by now.	7 years ago
Joxit	cceec0fe31	[Fix] Always apply geometric filter for placeholder	7 years ago
Julian Simioni	6bff132e69	Merge pull request #1171 from pelias/text-trim-whitespace Trim whitespace and quotes before checking text length	7 years ago
Julian Simioni	9a0f182fb2	fix(whitespace): Trim whitespace and quotes before checking text length Previously, our text sanitizer code did not trim whitespace before checking that the string was non-empty. This lead to strings consisting only of whitespace being treated as valid. Not all our downstream services (such as libpostal) accept whitespace-only input, so this causes a rather harsh error. This PR builds upon the code in https://github.com/pelias/api/pull/1170 and moves the trimming code above the nonEmptyString check. Now, a whitespace-only input string produces the normal error for empty input. Fixes https://github.com/pelias/api/issues/1158	7 years ago
Julian Simioni	435cfd2c4a	Fix linter error This snuck in in https://github.com/pelias/api/pull/1165	7 years ago
Julian Simioni	dd6f360c70	fix: Merge pull request #1165 from tiranno/pelias/pelias#670 Fix for Pelias/Pelias#670	7 years ago
Tyler Pedelose	be82c05c92	Moved includes check from lodash to native array in is_request_sources_includes_whosonfirst	7 years ago
Tyler Pedelose	85df33cdaa	Made is_request_sources_undefined more direct	7 years ago
Tyler Pedelose	df3e78ecdd	Uncomment predicate imports	7 years ago
Tyler Pedelose	52a66e1bc0	Added new predicates to aid placeholderGeodisambiguationShouldExecute	7 years ago
Tyler Pedelose	a0893cfb70	Removed extra comments	7 years ago
Tyler Pedelose	68c067ee8c	Fix for Pelias/Pelias#670	7 years ago
Julian Simioni	d139b3e08d	Merge pull request #1170 from pelias/trim_text trim whitespace and quotation marks from a range of natural languages	7 years ago
Peter Johnson	d38d4b1fa8	feat(text_sanitizer): trim whitespace and quotation marks from a range of natural languages	7 years ago
Julian Simioni	8f2f55c9a3	fix: Merge pull request #1162 from pelias/remove_focus_selected_layers Remove focus_selected_layers query view	7 years ago
Julian Simioni	561f07950b	Remove focus_selected_layers query view This query extends the standard focus query view with hardcoded layers for which the query applies. The intent was to apply the focus scoring only to non-admin areas, but the list of layers was already out of date, as it was missing streets. The query is fundamentally problematic with custom layers as well.	7 years ago

1 2 3 4 5 ...

2519 Commits (653cbf8770d983c6def841c3591287aaf7053761) All Branches Search

2519 Commits (653cbf8770d983c6def841c3591287aaf7053761)

All Branches