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

Corrección de appDir utilizado. #5

Open
wants to merge 3 commits into
base: developer
Choose a base branch
from
Open

Conversation

viktor992
Copy link
Contributor

Actualmente se está utilizando require.main.filename para obtener el punto de entrada a la aplicación. Esto podría ser incompatible con aplicaciones ejecutadas en iisnode (Windows/Azure) ya que require.main.filename retornaría un path similar a D:\Program Files (x86)\iisnode, (ver https://stackoverflow.com/a/20482383).

En este PR se aplican los siguientes cambios:

  • Se agrega la función getAppDir() para obtener el punto de entrada a la aplicación (ver utils/pathUtils.js), considerando que se está utilizando este módulo como dependencia.
  • Se reemplazan todas las ocurrencias de require.main.filename por el valor devuelto por getAppDir().

Se agrega utileria para obtener el path de entrada a la aplicacion.
De este modo se intenta agregar soporte a aplicaciones que estan
siendo ejecutadas desde iisnode en ambientes Windows/Azure.

Signed-off-by: Daniel Ughelli <[email protected]>
Se utiliza utils.pathUtils en lugar de require.main.filename en
las implementaciones existentes.

Signed-off-by: Daniel Ughelli <[email protected]>
@viktor992 viktor992 requested a review from javierpf November 18, 2018 21:51
Signed-off-by: Daniel Ughelli <[email protected]>
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

Successfully merging this pull request may close these issues.

1 participant