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

Sumar configuracion de variables de entorno #64

Conversation

calandrajose
Copy link
Contributor

LINK AL TICKET

Historia | Subtarea

DESCRIPCIÓN DEL REQUERIMIENTO

Al momento de crear SQSs utilizando https://www.npmjs.com/package/sls-helper-plugin-janis#sqs-helper se generan de forma automatica variables de entorno que contienen las urls de las colas, esto incluye al SQS/ propiamente dicho, DLQ. El problema que estamos teniendo es que al crear un SQS, dentro estamos generando variables de entorno para el SQS y sus respectivas DLQ. De estas variables de entorno, las unicas que se utilizan realmente son las de los SQS, lo que genera que tengamos n variables innecesarias que hacen que nos acerquemos al limite de 4kb que establece Amazon.

Solución

Modificar el paquete https://www.npmjs.com/package/sls-helper-plugin-janis#sqs-helperpara sumar la posibilidad de configurar si un recurso de SQS debe o no generar variables de entorno (generateEnvVars). La idea es que por default solo los SQSs guarden su url

DESCRIPCIÓN DE LA SOLUCIÓN

Se sumo para todos los tipos de SQS (main, DLQ y delay) la nueva propiedad generateEnvVars para poder definir si se deben crear las variables de entorno.
Por default sera true unicamente para las main, ya que son las unicas variables de entorno que efectivamente se estan usando en algun lado.

Ejemplo de uso


const { SQSHelper } = require('sls-helper-plugin-janis');

module.exports = SQSHelper.buildHooks({
	name: 'randomSQS',
	dlqQueueProperties: {
		generateEnvVars: true
	}
});

@jormaechea jormaechea merged commit ce64ae5 into jormaechea:master Oct 10, 2024
3 checks passed
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.

3 participants