enableProdMode stops AngularFirePerformanceModule?

[jonbcampos-alto]

Version info

"@angular/animations": "~9.1.0",
"@angular/cdk": "~9.2.0",
"@angular/common": "~9.1.0",
"@angular/compiler": "~9.1.0",
"@angular/core": "~9.1.0",
"@angular/fire": "^6.0.0",
"@angular/forms": "~9.1.0",
"@angular/material": "^9.2.0",
"@angular/platform-browser": "~9.1.0",
"@angular/platform-browser-dynamic": "~9.1.0",
"@angular/platform-server": "~9.1.0",
"@angular/router": "~9.1.0",
"firebase": "^7.14.6",

Other (e.g. Ionic/Cordova, Node, browser, operating system):
browser

How to reproduce these conditions

I have multiple environments: integration, testing, staging, production. AngularFirePerformanceModule doesn't seem to work in anything except for integration. The only difference between these environments is the environment variables, and the only one that I can think of that makes materially differences is the enableProdMode from production: true.

Failing test unit, Stackblitz demonstrating the problem

Steps to set up and reproduce

see above. 1 project, 2 env variables

Sample data and security rules
export const environment = {
production: false
}

vs

export const environment = {
production: true
}

Debug output

** Errors in the JavaScript console **
no console log errors

** Output from firebase.database().enableLogging(true); **
not db related

** Screenshots **
can't show screenshots of working performance for other reasons

Expected behavior

performance logged

Actual behavior

doesn't even show that the sdk is setup. contacted google support and they say they haven't gotten anything from this project (in production)

[jamesdaniels]

Perhaps the same as #2505, see my comment on that thread for a possible work around.

[vamsiambati]

Using the below code in app.component is registering all the logs in production mode
constructor(private perfSvc:AngularFirePerformance) { this.perfSvc.dataCollectionEnabled.then(dcp=>{ console.log('dcp enabled',dcp) }) }

[kyleabens]

@jonbcampos-alto I think I'm having the same issue as you. My performance monitoring was logging just fine in the Firebase dashboard up until a couple months ago. Now nothing is being sent over. Do we have to include PerformanceMonitoringService as a provider now since v6?

[jonbcampos-alto]

based on the thread it looks like the tree shaking is overly aggressive and stopping this functionality from working. until there is a fix I think the only option is to turn prod: false. which I don't recommend.

…

On Thu, Jul 16, 2020 at 10:51 AM Kyle Abens ***@***.***> wrote: @jonbcampos-alto <https://github.com/jonbcampos-alto> I think I'm having the same issue as you. My performance monitoring was logging just fine in the Firebase dashboard up until a couple months ago. Now nothing is being sent over. Do we have to include PerformanceMonitoringService as a provider now since v6? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2502 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKJW2HWGZMRTJIB7N4NPLALR34OX3ANCNFSM4N5COB7A> .

-- Jonathan Campos CTO and Vice-President Of Engineering E: jonathan@ridealto.com | W: ridealto.com P: 214-682-4070 900 Dragon St. Suite 100 Dallas, TX 75207 <https://www.facebook.com/ridealto/> <https://www.instagram.com/ridealto/> <https://www.linkedin.com/company/ridealto/> <https://twitter.com/ridealto?lang=en>

[madHorse54]

Here my solution:

1. in your app.module.ts import INSTRUMENTATION_ENABLED

import { AngularFirePerformanceModule, PerformanceMonitoringService, INSTRUMENTATION_ENABLED } from '@angular/fire/performance';

2. and set INSTRUMENTATION_ENABLED to true

providers: [ . . . PerformanceMonitoringService, {provide: INSTRUMENTATION_ENABLED, useValue: true}, . . . ],

[jonbcampos-alto]

tested @madHorse54 's solution. no effect.

[Splaktar]

It's possible that this was broken due to lint fixes similar to how 23e2f1e caused #2505.

The related lint fixes for Performance are in ae0ef48. I'm going to take a look and do some debugging there.

[jamesdaniels]

Yeah, looks like this is the case ae0ef48#diff-473350a8df09472fdfe71659bcae05daR36 the logic with the "default" was broken in that commit. If it's null, which is what an @Optional defaults to, it will disable metrics. I'm testing, will cut an RC with a fix for this and the analytics shortly.

[Splaktar]

Yeah, I saw that too but I ran into some issues trying to get the full cycle of build lib, yarn in sample project, ng serve in sample project, etc to see the changes. I haven't found any contributor docs that explain how to best do this development cycle for this project. It seems like I need to rm -rf sample/node_modules/;yarn after every time that I run yarn build to build the library.

[Splaktar]

Posted PR #2595 to solve this after I was finally able to verify the fix locally.

[Splaktar]

Here my solution:

1. in your app.module.ts import INSTRUMENTATION_ENABLED

import { AngularFirePerformanceModule, PerformanceMonitoringService, INSTRUMENTATION_ENABLED } from '@angular/fire/performance';

1. and set INSTRUMENTATION_ENABLED to true

providers: [ . . . PerformanceMonitoringService, {provide: INSTRUMENTATION_ENABLED, useValue: true}, . . . ],

In the meantime, I tried this workaround, but it didn't work for me. I also tried applying the fix in my node_modules/@angular/fire/ directory directly, but that also did not work for me. I guess that I'll wait for the next canary or release 😄

[jamesdaniels]

Will be fixed in 6.0.3 which should be up on NPM shortly.