search

bahnzumberg / zuugle-api

Public repositority for backend components of www.zuugle.at
https://www.zuugle.at
GNU General Public License v3.0
2 stars 3 forks source link

Limit query to necessary columns #32

Closed martinheppner closed 10 months ago

martinheppner commented 1 year ago

In Tours.js in line 794 "const query_con = knex('fahrplan').select().where({hashed_url: tour.hashed_url, provider: tour.provider, city_slug: city});" the select should be restricted to the columns, which are really used.

martinheppner commented 1 year ago

connectionsExtendedWrapper

Falsal commented 10 months ago

@martinheppner : Don't we use all 3 columns (hashed_url, provider and city_slug) to get any type of connection within the table 'fahrplan' ?

martinheppner commented 10 months ago

Is the question still open? The ticket is closed.