Loosen required versions dependencies

Is there any opposition to un-'pinning' the versions of both ember-cli-babel and ember-cli-typescript? I think adding a ^ is fairly safe given the stability and ubiquity of these two addons. It prevents people from having to do their own resolutions configuration to stay on the latest versions of these dependencies.

Hey @scottkidder I released v2.0.1 with up-to-date dependencies and unpinned version of ember-cli-babel and ember-cli-typescript. Cheers!

Thanks @charlesdemers. Cheers!


Rails 5  - devise omniauth callbacks validation issue

I'm trying to get omniauth working with my rails 5 app. Devise passwords are working fine (as they did when I was using rails 4).
I have an issue when I'm trying to add omniauth authentication methods to my existing user's account).
The error message says:

ActiveRecord::RecordInvalid at /users/auth/twitter/callback
Validation failed: User must exist

I have a user in my development db. My omniauth callbacks controller has:
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController

  def self.provides_callback_for(provider)

    class_eval %Q{
      def #{provider}
        auth = env["omniauth.auth"]
        @identity = Identity.find_by(provider: auth.provider, uid: auth.uid)

<EMAIL_ADDRESS>event: :authentication)) if @identity.present?

        if current_user.present?
          Identity.create_for_oauth(user: current_user, auth: env["omniauth.auth"])
          redirect_to user_path(current_user)
        else
          flash[:alert] = "You will need to register before signing in with you social media profile."
          redirect_to new_user_registration_url
        end
      end
    }
  end

  [:twitter,  :linkedin, :orcid].each do |provider|
    provides_callback_for provider
  end
end

My identities model has:
def self.create_for_oauth(user: u, auth: a)
    Identity.create!(user_id:      user.id,
                     provider:     auth.provider,
                     uid:          auth.uid,
                     accesstoken:  auth.credentials.token,
                     refreshtoken: auth.credentials.refresh_token,
                     name:         auth.info.name,
                     email:        auth.info.email,
                     nickname:     auth.info.nickname,
                     image:        auth.info.image,
                     urls:         auth.info.urls)
  end

I saw this article [(http://blog.bigbinary.com/2016/02/15/rails-5-makes-belong-to-association-required-by-default.html)]
I have a user, and I am trying to associate a new authentication method to that user's account. all of this worked fine in Rails 4.
Can anyone see what might be going wrong in rails 5?

@MincePie can you check whether current_user is persisted?

@MincePie Are you still experiencing this issue?


Add code coverage and coveralls

Pull request for Travis CI Test.
#255

According to the log https://s3.amazonaws.com/archive.travis-ci.org/jobs/172015221/log.txt,
some bugs and drawbacks will be fixed tomorrow.

Changes Unknown when pulling b5d685a90d24cc259cc67e396b252c5ac074195a on gangliao:coverage into ** on baidu:develop**.

Changes Unknown when pulling 73005c6a9536b6eac70ae38650e3f1f8c563e714 on gangliao:coverage into ** on baidu:develop**.

Changes Unknown when pulling dda876f0b32a3d4aa5af756ae4c170a0ee90cfd6 on gangliao:coverage into ** on baidu:develop**.

Does COVERALLS have account or key to login? If not, is that means anyone could post a fake CONVERALLS stats?

@reyoung  Coveralls needs to associate with github indeed and it also need our organization's authorization like travis.  All these thing already done three days ago.

Changes Unknown when pulling c0b650f72d7a463b0751d70265ddc38e774f3755 on gangliao:coverage into ** on baidu:develop**.

Changes Unknown when pulling e4a936d5d097bece3a01f459adf137aeb4aa3921 on gangliao:coverage into ** on baidu:develop**.


Filters for tasks

For example: I need show table for estimated time by people for task closed in this month or week

Hi miramir,
There is no straighforward way of doing that. I will however, share with you one instruction and one proposition. Please let me know what you think.
A. Instruction
The closest result can be achieved by the following steps.

Check "include closed"
Click the filter triangle next to "Closed" attibute to show filter dialog.
Click on "Select None".
Type in filter value such as "2016/01" to narrow down.
Click "Select All", then "OK".

The pivot should now show the number of issues closed within a designated range.
B. Proposition for improvement
What I can do is add functionality to automatically add attributes "Closed (y)", "Closed (m)", "Closed (w)", and "Closed (d)".
I assume this will allow you to filter quicker. You won't be able to filter like "last 30 days", but you can jump to "January of 2015" or "Week 4 of 2016". If you want to see a weekly summary, you just neet do drag "Closed (y)" and "Closed (w)" to the horizontal attributes.
I did not implement this because there are many date attributes, and expanding year, week, and other attributes to all date field could clutter the available attribute panel.
So, i guess the choice is

Add additional date attributes to all date fields.
Add additional date attributes to "Closed".
Do nothing.

Hope this helps.

Oh, and for the estimated time by people, drag "assigned to" to vertical attribute, select "Sum" as an aggregator, and "Estimated time" as aggregated field.


Add another pass to check for attributes

This would move most of the FFI function checks from runtime to compile time. It is also a superset of #22

This was meant for the case where if a function has the attribute #[builtin(sha256)] for example, then the compiler would check that the function signature matched what we would expect from a sha256 function.
When we call blackbox functions, this is triggered, so I'm marking this as no longer planned, since that is good enough


#374: light-background -> content-background

Reduce the size of the PR for #374

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 94.086%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6120

Relevant Lines:
6277

💛  - Coveralls


⚠ DesiTorrents has degraded performance

In 12f005e, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 4682 ms

Resolved: DesiTorrents performance has improved in 972b6b1.


WIP: Generating invalid data

Resolves #65 and #66

[ ] properly filter data, generated by not_from_schema - should be dictionaries and for path probably not arrays or objects
[ ] high level tests for not valid data
[x] CLI option for invalid data generation
[ ] Support in CLI / Runner
[ ] Support in lazy schemas
[ ] changelog entry
[ ] docs

Closed in favor of #831


[test] Add smoke test and loader tests

:tada: tests!
One basic smoke test just to make sure things are compiling as expected, and a couple of loader-specific unit tests.
@TodayTix/web can take a look at this, if they'd like. I'll wait for it to at least be a workday before merging it 😉

Man, that yarn.lock file is gonna make my contribution numbers look awesome


feat: add outerHost config property

Fix #48

:tada: This PR is included in version 3.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Sourcemaps with urlencoded chars not loaded

Link to bug demonstration repository
I have setup a mini example of the issue:
https://github.com/kazkansouh/nyc-urlencode-bug
Expected Behavior
I expect nyc to correctly handle file paths that contain url encoded characters.
Observed Behavior
$ npx nyc node dist/\{id\}.js
Transformation error for /dev/shm/nyc-urlencode-bug/dist/{id}.js ; return original code
An error occurred while trying to read the map file at /dev/shm/nyc-urlencode-bug/dist/%7Bid%7D.js.map
Error: ENOENT: no such file or directory, open '/dev/shm/nyc-urlencode-bug/dist/%7Bid%7D.js.map'

If I create symlinks from dist/%7Bid%7D.js.map to dist/{id}.js.map nyc runs perfectly ok. I notice that {id}.js contains the url already encoded so perhaps its missing a decode call?
Environment Information
$ npx envinfo@latest --preset nyc

  System:
    OS: Linux 5.4 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz
    Memory: 290.67 MB / 15.37 GB
  Binaries:
    Node: 10.19.0 - /usr/bin/node
    npm: 7.20.0 - /usr/local/bin/npm
  npmPackages:
    nyc: ^15.1.0 => 15.1.0
    typescript: ^4.3.5 => 4.3.5

Anyone has any ideas here? I have the same issue exactly.
Everything else is working (if I don't have such files, it all works fabulously)
@kazkansouh did you find a resolution?


save env vars to file more safely

Before, we exported environment variables to a file using the command env > jenkins-env. This works until you have spaces in the value of the exported variables.
With an env variable like ghprbPullDescription=Github pull request the whole thing will break when reading the jenkins-env file back in. We have seen this job for example where we've encountered this error:
++ ghprbPullDescription=GitHub
++ pull request
jenkins-env.clean: line 14: pull: command not found

This PR fixes the way we save environment variables by using declare -p instead of plain env. A variable like the above will be stored as declare -x ghprbPullDescription="GitHub pull 111 request" and can safely be read in using source.
For more information on this type help declare in a terminal or look at  this post.
All consumers of the jenkins-env file should be fine if they use source to read the values back in.
Don't do something like sed 's/^/export /g' on the jenkins-env file. If you want to filter out variables, you can so like so:
cat jenkins-env | grep -E '(JENKINS_URL|GIT_BRANCH|GIT_COMMIT|BUILD_NUMBER|ghprbSourceBranch|ghprbActualCommit|BUILD_URL|ghprbPullId)=' > ~/.jenkins-env
source ~/.jenkins-env

[test]

Closing this PR, this effort will be tracked from https://github.com/openshiftio/openshiftio-cico-jobs/issues/580


Stack run.Dockerfile default ENV PORT overrides buildpack's env.launch defaults

Problem
It looks like the google CNB stack sets a default PORT=8080 in its Dockerfile. Running docker inspect on an image similarly shows that Config.Env has PORT=8080.
This makes it difficult to customize the default value of PORT at runtime, presumably because it gets set before the defaults in e.g.

a custom buildpack's launch.env/PORT.default
the app's .profile

are considered.
Example: .profile
If I have a .profile with
export PORT=${PORT:-5000}

then the runtime value will remain at 8080, but
export PORT=5000

overrides the value.
Example: launch.env/PORT.default
Say I have a custom buildpack at $custom_buildpack_path with a trivial bin/detect, and following bin/build and buildpack.toml, respectively
# bin/build

LAYER_DIR="$1/override-port"
ENV_DIR="$LAYER_DIR/env.launch"
mkdir -p "$ENV_DIR"
echo -n 5000 > "$ENV_DIR/PORT.default"
echo "launch = true" > "$LAYER_DIR.toml"

# buildpack.toml

api = "0.2"

[buildpack]
id = "com.example"
name = "Set a default runtime PORT environment variable"
version = "0.0.1"

[[stacks]]
id="google"

and prepend the custom buildpack google's set of buildpack
pack build test-default-port \
  --builder gcr.io/buildpacks/builder:v1 \
  --buildpack $custom_buildpack_path \
  --buildpack from=builder

then
docker run -it --rm test-default-port 'sh -c "echo $PORT"'

outputs 8080 as opposed to 5000.
Contrast that to an image built with gcr.io/paketo-buildpacks/builder:base, where the docker run command outputs 5000.
Proposal
Get rid of run.Dockerfile and use <layers>/<layer>/env or <layers>/<layer>/env.launch instead.

I don't think this is bug. Your env var isn't getting set when you docker run because you are bypassing the buildpack launcher: https://buildpacks.io/docs/app-developer-guide/run-an-app/#no-launcher
I suspect if you did this it would work:
docker run --rm --entrypoint launcher test-default-port -- sh -c "echo $PORT"


[Bug]: check_extra79 does not check WAF settings

What happened?
The remediation for rule extra 79 is "Ensure the load balancer should be publicly accessible. If publicly exposed ensure a WAF ACL is implemented"
How ever it does not seem to check for WAF ACL implementation
How to reproduce it
Steps to reproduce the behavior:
run rule
Expected behavior
Load balancer that implemented WAF ACL should pass this rule

Hi @tayivan-sg, did you tried with Prowler v3? Right now v3 is available either pip install prowler-cloud or using master branch in thew Github repository.
Please, test it and let us know.
Thanks for using Prowler!

Hi @tayivan-sg, we've reviewed the Prowler v2 check_extra79 and the recommendation is right. This check looks for Public ELBs and raises a FAIL if it's publicly exposed.
The recommendation for this check is that if your ELB has to be Public it is recommended to put a WAF ACL in front of it.
Thanks for using Prowler!


#pywhatkit

I don't know  why it's taking that much can anyone tell me deal with this
ya, I am using MC os.

Is whatsapp web taking time to load?

no
there was an issue with the code that took more than 24 hours to run the script so now I am using pwk.sendwhatmsg_instantly


some changes to the htmls

There are conflicts with the work @shanishapp made in single_comment.html. Shani's look better, @asaf-kali will merge.

Three are conflicts with the work @avihai-huji done in the sign up and login pages. I think the best strategy is after this merge, @avihai-huji will merge his new code.

There is no need for two signup pages, this is a duplication and needs to be solved some other way.
Anyway, I disabled facbook for now - it makes development process slower, we can add this feature later.


Build an obvious reporting tool for PDF errors

It's expected that early in the development process, and with such a wide audience to test, that a lot of PDFs will throw errors.
We should build a popup or box that displays when a PDF fails with the error and instructions on how to report it (what is the process to report a bug? An email address?). This will hopefully increase reporting rate and visibility into potential errors.

As I was troubleshooting a problem PDF last night, I was thinking same thing! I think email is probably the simplest. What do you think? I could just add any issues here when I get them.

The email is definitely simplest for starting out, and I think a lot of people will be supportive. There are fun tools like Rollbar that do automated issue management, but that also requires that we have an API setup (which I don't believe we have today, although I haven't really looked!)
I would create a new email address that isn't your personal one. Otherwise, I think it's great!

Agreed! No API yet. I do have an email setup for PP. It's<EMAIL_ADDRESS>I should really link it up to my github account!

Ok, so I created a dialog that displays render and load errors, however, I'm a little concerned that it is too touchy. The dialog would display even when the pdf successfully loaded, though it's possible some of the pdf did not load correctly. I tried several PDFs tonight from several pattern companies, and almost all of them loaded without issue.
I will still add a PR, but we can review and decide whether this is the solution we want.

Based on our discussion in #12, this is going to change. I wrote out helper classes for modals and buttons, so I don't want to entirely abandon this. I will stash the branch and see what we come up with on the rendering side of things.


Lower VMVX linalg ops to library calls.

This is a slash and burn that:

Creates an opinionated small set of tiled math kernels that we implement via library calls.
Lowers linalg->vmvx library calls for supported kernels.
Implements the world's worst loopy versions of these in C (enough to be correct and show that it works).
Fixes a number of rough spots in the various compilation paths for memrefs and VM compatibility.

The intent is to land this eventually behind a flag and to ultimately enable by default once we have converged on the right set of kernels. Since these library calls are cutting in after tiling/distribution, they are operating on small tiles and are externally parallelized. As such, they can be thought of as micro-kernels and should be much simpler than their full featured cousins. In addition, since they are mostly non-polymorphic and should be amenable to jitters, hand coding, etc.
On the fully_connected case I was iterating on (%result = "tosa.fully_connected"(%0, %1, %2) : (tensor<512x384xf32>, tensor<128x384xf32>, tensor<128xf32>) -> tensor<512x128xf32>), even with terrible loopy C kernels, this sped benchmarks of VMVX up by between 100-200x. Note that due to reasons, the above also includes a leading transpose, which is not yet lowered to library calls, so the interpreter is still doing quite a bit. Further note that there are additional peephole optimizations that I haven't done yet to eliminate the fill, fold the add and set flags on the matmul to disable accumulation. So plenty to do from here.
This PR isn't ready to submit yet: it is a slash-burn through a lot of layers and we'll be working to pick bits out and finish it.

looking great! I can't speak to the linalg/memref portions like mahesh could, but everything else looks fine for the first iteration - as we scale out the ops we'll probably settle on a consistent style/helpers for conversion/utils at runtime and I think this is a good starting point.


proposal: cmd/go: Reserve several simple package path prefix to the user, never use it in the standard library

From https://github.com/golang/go/issues/34068#issuecomment-536015992

Package paths that do not start with a domain name are in general reserved for the standard library. That rules out dotless names like app and make, especially in module mode.

That's a strange rule for me.And it is not compile error, so I do not notice that since now.
My private projects have used a lot of simple package path like GOPATH/src/make/run/main.go or GOPATH/src/make/tmp/main.go to simple build and run of those package with a private command line tool.xxx make run will build and run the package make/run. I need to set GOPATH first, if I want use go run before go.mod, So I write my own command line tool during go1.4 to not set the GOPATH before every call of go run.
Since I do not want to share the main package code with the world, why do I need the long domain name to type or write a script with another language every time I need to build/run my main package in my project?
A another reason to not use a domain name in package path is that the package path will go to the finally binary file. If I want the binary file have as less keywords as I want, i will not use a domain name in package path.
So I suggest golang leave several simple package path prefix to the user. And have a test to check that it never appear in the standard library. The main use case should be the exist
suggest for several simple package path prefix for the user:

GOPATH/src/a/
GOPATH/src/make/
GOPATH/src/config/

reason:

GOPATH/src/a/ for shorter package path in the private project, one letter should be the shortest one of them.
GOPATH/src/make/ for main package list in the private project, It is easy to list them and call them as they all exist in one directory. It is used heavily in my private project(100+ per project), another name should be ok for me. (I need rename all of them before I upgrade to the golang version that use it.)
GOPATH/src/config/ for packages that is used for project level config in golang code. Write config in the golang code is easy to deploy the finally binary into server or client.

I think GOPATH/src/vendor should also reserve to the user, there are several projects that can not compile because of this (https://github.com/golang/go/issues/34068), but it has already used in go1.13
The GOPATH/src/internal do not need reserve to the user, as the private project is already internal.

I doubt the standard library is actually going to add a make or config package (and thus break your code) any time soon. If you nevertheless want to hedge against that, you could just switch to a short dotted name like ma.ke or con.fig, which doesn't require any changes to the toolchain.

@bronze1man, note that in module mode, the module path does not need to appear in the file tree at all, and you can refer to paths within that module using the prefix ./.
So the benefit in terms of typing saved is already (mostly) achieved simply by switching to modules.

Thanks for your reply.
Looks like I can just use a random prefix in the package path or rename my package when GOROOT use it if I want to type less in the GOPATH.
Reserve package path looks a bad idea for golang.


draft

@sstrubberg What was this for?

Whoops! I was just messing around with sub-issues and tasklist in GitHub. Feel free to use this however you see fit or we can just close it.


Id_token renew issue

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Browser:

[ x] Chrome version XX
[ ] Firefox version XX
[ ] IE version XX
[ ] Edge version XX
[ ] Safari version XX

Library version

Library version: 1.0.1

## Current behavior

I use msal.js in my vue SPA for authenticating the user with Azure Active Diretory and sending the idToken as authorization header to an API.

 Everithing goes well when the user opens the app for the first time, i receive only the id_token:
 this.loginRequest = {
        scopes: ["openid", "email"],
        prompt: 'select_account'
    }
this.msalnstance = new Msal.UserAgentApplication(this.config)

this.msalnstance.loginPopup(this.loginRequest)
            .then(function (loginResponse) {   //here i take the id token from response and the user email  }

But after the id token expires and i am trying to renew it, i run into issues. I saw in the acquireTokenSilent method definition that if i send the clientId as scope, then the idToken will be refreshed. First i'll show you the code i use:

  this.idTokenRequest = {
        scopes: [my_clientId_value],
        authority: my_authority_value,
        prompt: 'none'
    }

 this.msalnstance.acquireTokenSilent(this.idTokenRequest)
                .then(function (idTokenResponse) {
                   // here i try to get the user's email again, but i cannot find it in the response
                 }

This setup brings a value for access token, sometimes is the same as id_token, even if the value for "token_type" in response is "id_token".
Sometimes it brings a different value for access token (with exp date in the future), while id_token is not renewed.
Sometimes the id_token is renewed, but when that happens, it doesn't bring also the "email" value and i cannot add extra scope for that, beside client id value.
Other errors i got when trying to refresh the id_Token are:
- Nonce is not matching, Nonce received:  [value], nonce expected : [value]
- set-Cookie header is ignored in response from url https://login.microsoftonline.com/tenentid...... Cookie length should be less than 4096 characters
## Expected behavior

I expect that acquireTokenAsync renew only the id token, not the access token, when i pass clientId in the scopes and also I need a way to bring email in the renewed id token.
Also, because i need only the id token, i shouldn't be constrained to set to "true" in app registration Manifest oauth2AllowImplicitFlow, but only oauth2AllowIdTokenImplicitFlow.

## Minimal reproduction of the problem with instructions

@sameerag   / @negoe  cc
I think there may be some confusion in usage here.  AcquireTokenSilent() is specifically for acquiring an accessToken, not an id token.  Id tokens are obtained by logging a user in, or by calling GetAccount() (if already logged in) access token renewal is done.

This is from the acquireTokenSilent method (msal.js 1.01.)definition:

To renew idToken, please pass clientId as the only scope in the Authentication Parameters

@returns {Promise.} - a promise that is fulfilled when this function has completed, or rejected if an error was raised. Returns the {@link AuthResponse} object
acquireTokenSilent(request: AuthenticationParameters): Promise;

On my local environment the id token is renewed, but i need to add also the "email" scope when i call acquireTokenSilent , so i can take the value from the id token. This is how i identify the logged in user in my API, by sending the id token in the authorization header from SPA app to API and here i validate it and get the email claim value.

the current method to get an id token, with preauthorized scopes is loginPopup or loginRedirect methods.  You can get an id token with acquireTokenSilent by passing the clientId in scopes, but then I believe you can not add email to it.
Would something like this work
acquireTokenSilent(requestWithClientId).then(() => {
  return acquireTokenSilent(requestWithEmailScope)
}).then(() => {
 return acquireTokenSilent(requestWithClientId)
}).then(response => {
    //  response has idTokenwithClamsYouWant?
});

I think we definitely need to support a simpler way here, but curious if this works.

@madatan  ping to try this

Thanks DarylThayl for suggstion. Although I don't think this should be the solution to the issue, I'll try it.
I managed to find a workaround for email by using another field value from id token, but now the biggest issue for me is that sometimes it doesn't renew the id token when calling acquireTokenSilent(requestWithClientId), instead it renews the access token. :(

@madatan I am a little confused on the renewal of access Token comment. We have a check in our code where we renew idToken specifically in the acquireTokenSilent() call if the scope passed is clientId:
if(request.scopes && request.scopes.indexOf(this.clientId) > -1 && request.scopes.length === 1) {
    // App uses idToken to send to api endpoints
    // Default scope is tracked as clientId to store this token
    this.logger.verbose("renewing idToken");
    this.renewIdToken(request.scopes, resolve, reject, account, serverAuthenticationRequest);
} else {
    // renew access token
    this.logger.verbose("renewing accesstoken");
    this.renewToken(request.scopes, resolve, reject, account, serverAuthenticationRequest);
}

Can you share your sample code where you get an accessToken renewed instead of idToken?
We also have token renewals documented here, in case if it helps your case.

Hi,
Here is the code:
this.loginRequest = {
        scopes: ['openid', 'email'],
        prompt: 'select_account',
        authority: process.env.VUE_APP_AUTHORITY
    }

    this.config = {
        auth: {
            clientId: process.env.VUE_APP_CLIENT_ID,
            redirectUri: process.env.VUE_APP_CALLBACK_PATH,
            authority: process.env.VUE_APP_AUTHORITY

        },
        cache: {
            cacheLocation: 'localStorage',
            storeAuthStateInCookie: true
        }
    }

    this.msalnstance = new Msal.UserAgentApplication(this.config)

    this.renewTokenRequest = {
        scopes: [process.env.VUE_APP_CLIENT_ID], // here process.env.VUE_APP_CLIENT_ID is my Azure AD application id value
        prompt: 'none',
        authority: null, // I tried also without setting authority to null
        account: this.msalnstance.getAccount() // this one and the authority value I added because of another thread from git with another id token renewal issue
    }

//Method for user login
login() {
    let self = this
    return new Promise((resolve, reject) => {
        let account = this.msalnstance.getAccount(),
        hasAccount = !!account
        if (hasAccount) {
            this.msalnstance.acquireTokenSilent(this.renewTokenRequest)
                    .then(function (response) {
                      // here I get the response.idToken.rawIdToken
                        resolve()
                    })
                    .catch(function (error) {
                        console.error(error)
                        if (error.errorMessage.indexOf('interaction_required') !== -1 || error.errorMessage.indexOf('no user is signed in') !== -1 ) {
                            self.interactiveLogin()
                                .then((email) => {
                                    console.log('interactive login email', email)
                                    resolve(email)
                                })
                                .catch((error) => reject(error))
                        } else {
                            reject(error)
                        }
                    })
        } else {
            this.interactiveLogin()
            .then((email) => {
                resolve(email)
            })
            .catch((error) => {
                console.log(error)
                reject(error)
            })
        }
  })
}

//Method used for checking if user is logged in and also for id token renewal
userIsLoggedIn() {
    let self = this
    return new Promise((resolve, reject) => {
        let account = this.msalnstance.getAccount()

        let hasAccount = !!account
        if (hasAccount) {
            this.msalnstance.acquireTokenSilent(this.renewTokenRequest)
                .then(function (response) {

                   //// here I get the response.idToken.rawIdToken
                    resolve()
                })
                .catch((error) => {
                    console.log('Error in userIsLoggesIn method')
                    if (error.errorMessage.indexOf('interaction_required') !== -1 || error.errorMessage.indexOf('no user is signed in') !== -1) {
                        self.interactiveLogin()
                            .then((email) => {
                                resolve(true)
                            })
                            .catch((error) => reject(error))
                    } else {
                        reject(error)
                    }
                    console.log(error)
                    })
        } else  {
            resolve(false)
        }
    })
  }

Hi,
I'm also curios how can we renew the id_token with email claim in it?
@madatan have figured this out?

Is there a status update for this issue?  We are actively waiting on the solution.

@sameerag  do you have anything you can add here on this issue?
Does the claims object you added here add this?

This problem should be fixed in recent releases, please upgrade to<EMAIL_ADDRESS>and test again. Comment if this is still a problem.

@jmckennon I updated to<EMAIL_ADDRESS>and still have the issue, is there a new flag like forceRefresh that I need to use to force an Id_token renewal?
forceRefresh flag still only refreshes the access token when used in acquireTokenSilent.

@arian2ashk Can you please elaborate on the issue you are seeing, and how you are using the library?

@jasonnutter the setup we have is an API and a react app that talks to the API. the API manages users and roles and permissions and it uses Azure AD for providing the identity. we have different ways of authenticating in our API for different customers. if there is a token in the header meaning the customer has a token provider, it will validate the token and then use the info in the token to retrieve the user from the database.
When using AzureAD since token issued for Microsoft Graph cannot be validated we use the Id_token to validate the token and we use the access token to retrieve some user's info from Graph on the API side. because of this we need both access token and id token. on the react side we have a provider that will authenticate and provide a token on the context, so underlying components can pick up the token and attach it to the header of the request they make. when a user has been authenticated and tokens exist a token renewal task will be scheduled that will renew both access token and Id token before the expiration and change the tokens in the context. here is the simplified code that does the authentication and renewal:
`
login = (onAuthenticationSuccess, onAuthenticationFail, authParams = {scopes: ['openid', 'email'], forceRefresh: true}) => {
this.myMSALObj
.acquireTokenSilent(authParams)
.then(authResult => {
if (authResult && authResult.accessToken && authResult.idToken) {
/** schedule a token renewal based on when the current token will expire /
this.scheduleRenewal(authResult.idToken.expiration);
onAuthenticationSuccess(authResult.idToken.rawIdToken, authResult.accessToken);
}
})
.catch(e => {
try {
/ No session exists, User needs to login. Calls loginRedirect to redirect to AzureAD login page */
this.myMSALObj.loginRedirect(authParams);
} catch (error) {
onAuthenticationFail();
}
});
};
scheduleRenewal = (expiration, onAuthenticationSuccess, onAuthenticationFail) => {
var expiresIn = expiration * 1000 - new Date().getTime();
//renew the token when 90% of the time has passed to make sure token is renewed before expiring
var delay = expiresIn * 0.9;
if (delay > 0) {
this.tokenRenewalTimeout = setTimeout(
that => that.login(onAuthenticationSuccess, onAuthenticationFail, {scopes: ['openid', 'email'], forceRefresh: true}),
delay,
this
);
}
};
`
We need both tokens to be renewed and the forceRefresh flag only renews the access token. I know our solution may not be the best solution or following the standard way but it worked for our other token providers, and since our solution needs to be generic enough to work for all our customers with different token providers, we needed to diverge from the standard way. Thanks in advance for helping.

@arian2ashk Thanks for the info. A couple of thoughts:

Generally speaking, we do not recommend using ID Tokens to validate your own web API (and instead use access tokens with custom scopes). Since your web API appears to be a simple pass-through proxy to Graph (and you already have a token for Graph), I would consider removing that requirement and just pass the access token, since Graph will return an error if the access token is not valid.
MSAL takes care of renewing tokens if they are expired (or about to expire), so you should not need to have this logic in your app. Our guidance is to call acquireTokenSilent right before you need an access token, and rely on MSAL to either return you a cached token, renew your token, or throw an error prompting you to call one of the interactive methods.

Here are the steps I would implement:

When your app wants to make a call to your web API (which will call Graph), check to see if the user is currently logged in.
If not, call loginRedirect or loginPopup.
If they are logged in, call acquireTokenSilent.
If that succeeds, use the access token from the response and send it to your web API.
If it fails, check that the error is a user interaction error, and if so, call acquireTokenPopup or acquireTokenRedirect. One reason this could happen would be if there is not a valid token in the cache and the user no longer has an active AAD session. Calling one of the interactive methods will give you a new access token and new ID token, in that case.
Once that is complete, use the access token from the response and send it to your web API.

Let me know if that makes sense, and if you have further questions.

@jasonnutter Thanks for the feedback, It helped a lot, I managed to only use the access token for the authentication by using custom scopes. The only question I have right now is that why does MSAL loginRedirect and acquireTokenSilent require implicit grant to ID tokens. I was expecting now that I am not using the ID token I should be able to remove the implicit grant to ID tokens.

@arian2ashk You're welcome!
MSAL requires an ID token because it requires context about the user, which the ID token provides.


Bug on repeatable qualdrop value

Describe the bug
trying to delete a qualdrop fields with different metadata generates an error
To Reproduce
Steps to reproduce the behavior:

start a new submission with a qualdrop value field (e.g a publication has Identifiers field)
enter a new identifier
save submission
click add more to add a new identifier
select a different identifier than the one saved before
enter the new identifier
save the submission
click on the Trash icon to delete the last identifier added
save the submission an error message should be displayed

Expected behavior
The delete operation should be saved successfully

@tdonohue it takes 1hour to fix, I can include fix directly in the current #1201

@atarix83 : Sounds good. Moving onto the 7.0 board. Feel free to fix in 1201


Accessibility

Is there any plans to enhance Slider Pro with keyboard input?
As part of my organization, we must adhere to Section 508 and W3C accessibility guidelines. And the only glitch to using Slider Pro is it doesn't have any keyboard inputs for switching between slides and accessing the content once inside a slide.

Hi. It might not be all that you need, but you can actually navigate between slides by using the left and right arrow keys, and also navigate to links by hitting the Enter key.
Cheers!

Hello,
It would be great if it was possible to disable only the "hitting Enter key to follow the link" part.
I'd like to keep the arrow keys navigation, but right now, I can't submit a form which is on the same page (like a search engine) with the "Enter Key".
Should I open a new issue?
Thank you for your work btw.

Hi,
You can set the keyboardOnlyOnFocus option to true, so that the slider will be navigable by keyboard input only if the slider has focus. Or you can disable this module, by setting keyboard to false and then add this code:
$( document ).on( 'keydown', function( event ) {
	if ( event.which === 37 ) {
		$('.slider-pro').sliderPro('previousSlide');
	} else if ( event.which === 39 ) {
		$('.slider-pro').sliderPro('nextSlide');
	}
});

You're welcome!

Thank you for the tip!


Fix sidebar on mobile (beta)

Fix sidebar on mobile. On iPhone it cuts sidebar when scrolling.

https://github.com/user-attachments/assets/1886e293-0a83-426f-9b17-ac94d8599769

Sidebar width 250px instead 210px.

https://github.com/user-attachments/assets/f78bdae4-2204-4d33-9982-fc69400da721

@bwp91 fixed

something is still not quite right in mobile view.
the menu icons appear before the menu has fully expanded and it looks a little weird

https://github.com/user-attachments/assets/98092e09-8120-44c1-ae86-eb4ba3a60c41


git add autocomplete works bad after first executing

Here is an example: https://cl.ly/3c1A1P2g1d3m

+1 on this issue.
I have the same behaviour.


Accessing wrong key in token verification util on password reset route

Think you meant to decoded_token['sub']?
https://github.com/tiangolo/full-stack-fastapi-postgresql/blob/490c554e23343eec0736b06e59b2108fdd057fdc/{{cookiecutter.project_slug}}/backend/app/app/utils.py#L104

Yes, and a pull request already exists: https://github.com/tiangolo/full-stack-fastapi-postgresql/pull/172

Sweet thanks, closing this.

Thanks for the help @kurtrottmann! 🚀
Thanks for closing this one @rkrn ☕

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


How to remove Consul

Why after applying consul via Terraform to AWS and later removed, I still get the consul service back up?
module "consul" {
  source = "hashicorp/consul/aws"
  version = "0.3.3"

  aws_region  = "us-east-1" # should match provider region
  num_servers = "3"
}

How to remove as this is running and I don't want it
$ terraform destroy : Destroy complete! Resources: 5 destroyed.
But I had to manually delete each of the consul services on AWS console ???

Just following from tutorial:
https://learn.hashicorp.com/terraform/getting-started/modules
But now Consul cluster cannot be removed, and terraform apply :
Error: Error applying plan:
1 error(s) occurred:

aws_vpc.main: 1 error(s) occurred:

aws_vpc.main: Error creating VPC: VpcLimitExceeded: The maximum number of VPCs has been reached.
status code: 400, request id: 58f95039-3010-4478-9fa3-398e2d240934


-- nk. Added checkout for variable ConqueTerm_SessionSupport exists

Hope this helps. Let me know if this works and you'd rather is use master branch PR -- cheers. krishnan

LGTM, btw what is nk?

Hi,
Great. nk is my initials; just a string to mark any changes i've made to
external repos to make search easier later.
(Have removed the comment marking original code and pushed to my repo;
looks like you can merge pull this?)
Thanks,
-- k
On Friday 30 December 2016 12:09 PM, Kentaro Wada wrote:

LGTM, btw what is nk?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wkentaro/conque.vim/pull/3#issuecomment-269737858,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASBXjgcn4dx1UHUCIVQ8cigRHpxCPgdks5rNKcHgaJpZM4LXZpl.

--
CEO
gai Technologies Pvt Ltd
Nandan Ghar, Lower Barol
VPO Dari, Dharamshala,
Himachal Pradesh - 176215
INDIA
Ph: +91 9816-088-451 (Office)

Great. nk is my initials; just a string to mark any changes i've made to
external repos to make search easier later.

Why don't you use git log --author='YOURNAME' or filter by author on GitHub?
I think -- nk prefix is a bit confusing and would like to remove it. I can remove it with Squash and Merge button at merging this PR.

Hi,
Please go ahead and remove '-- nk'. Let me know if you want me to do
another push or you can do it yourself.
Great plugin, BTW!
Thanks,
krishnan
On Friday 30 December 2016 01:17 PM, Kentaro Wada wrote:

Great. nk is my initials; just a string to mark any changes i've made to
external repos to make search easier later.

Why don't you use |git log --author='YOURNAME'| or filter by author on
GitHub
https://github.com/wkentaro/conque.vim/commits/master?author=wkentaro?
I think |-- nk| prefix is a bit confusing and would like to remove it. I
can remove it with |Squash and Merge| button at merging this PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wkentaro/conque.vim/pull/3#issuecomment-269742355,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASBXuIxSsvQlzY2fjsYXk5b41-JKLrjks5rNLcjgaJpZM4LXZpl.

--
CEO
gai Technologies Pvt Ltd
Nandan Ghar, Lower Barol
VPO Dari, Dharamshala,
Himachal Pradesh - 176215
INDIA
Ph: +91 9816-088-451 (Office)

Thanks.


update wiki docs

add developers guideline
add database/connection.py documents credential_events, honeypot_events_data, honeypot_events, network_events
add module specific document (ics, http, etc)
....

@Ali-Razmjoo : I will update it by this weekend

I would like to work on this issue if its still up @dhirensr @Ali-Razmjoo

I think @dhirensr is taking care of it, right?

@amanjiofficial : I am working on it.

@Ali-Razmjoo : we can close this ticket as all the documentation mentioned is done!

thanks @dhirensr


Still High rent and terrain color overview

Frist of all, I want to say that this is an essential mod for cs2, no one should play without it. I used it before pdx mods and it was flawless, as soon as I saw it today published, I subscribed and wanted to start playing cs2 again.
But this version is different (game version is different as well), I loaded a city which I build with previous version of Land Value Overhaul and almost immediately High Rent complaints everywhere (last time I played, there wasn't even a single High Rent complain).
Also when you are in land value info mod, fertile land, telecom covarage, air and noise pollution, there is no color on the ground. I tested this whit only Land Value Overhaul mod, every other mod disabled.

Go to the LandValueOverhual.log and see if building upkeep get modified, if not, it's not correctly loaded, and it is due to PDXMods bugs.
The land value info is just not there for modded saves from previous versions, no matter with or without any mod.

Did you updated the mod, or did pdx finally loaded as it should? I loaded up a game now, some old city with a lot High Rent issues, they all disappeared in a min. Unfortunately in info those info panels, there is still no color on the ground.

If you switch from BepInEx to PMODs, you need to remove EVERYTHING related to it. And I don't know if they can be used simutaneously, since I have experienced weird mod loading behaviors using both.

If you switch from BepInEx to PMODs, you need to remove EVERYTHING related to it. And I don't know if they can be used simultaneously, since I have experienced weird mod loading behaviors using both.

Yes, on 25th March I reinstalled the game and removed all leftover folders.

If the high rent issue disappeared quickly, then it should'nt be a problem.
The LV info panel just won't show up. I have tried a 1.0.19f1 save on 1.1.0f1, without ANY mod, and also no color on the ground.

The new modded LV mechanism is indeed slightly different from 1.4.3, and it need more testing.
If at your convenience, could you report what kind of zones tend to yield high-rent warnings. If it's the low-density residentials, what is the average rent and maximum rent (via the developer mode)?

I was talking about this info view
With LVO
-Natural Resourses

-Telecom Covarage

Without the mod
-Natural Resourses

-Telecom Covarage

I just created a new playset to test these info views, and immediately found the building upkeep being not correctly loaded.

And if I change its load order to before all downloaded save games, it immediately worked.

Did you updated the mod, or did pdx finally loaded as it should? I loaded up a game now, some old city with a lot High Rent issues, they all disappeared in a min. Unfortunately in info those info panels, there is still no color on the ground.

I loaded the same city, this time hing rent stayed.

I guess it's a problem with PDX mods, or load order.

No, I'm currently working on the info view issue. It is easy to solve but so hard to make PDXMods correctly load LVO in WIP.

Did you updated the mod, or did pdx finally loaded as it should? I loaded up a game now, some old city with a lot High Rent issues, they all disappeared in a min. Unfortunately in info those info panels, there is still no color on the ground.

I loaded the same city, this time high rent stayed.
I guess it's a problem with PDX mods, or load order. Just need to restart a game couple of times until LVO starts working correctly.

The low-density upkeep should be around 16 or something similar. If not, LVO is not correctly loaded.

If I totally remove everything related to PDXMods (removing .cache and .pdxsdk folders under ), never log in your PDX account in-game, and install the mod by copying mod content to $user_directory$\AppData\LocalLow\Colossal Order\Cities Skylines II\Mods. The mod is guaranteed to be correctly loaded.

If I totally remove everything related to PDXMods (removing .cache and .pdxsdk folders under ), never log in your PDX account in-game, and install the mod by copying mod content to userdirectory\AppData\LocalLow\Colossal Order\Cities Skylines II\Mods. The mod is guaranteed to be correctly loaded.

I tried to restart a game 3 times in the row and it loaded LVO no problem.  Maybe it was just luck. I guess this is a solution for now, I will install it manually as you said, this is most important mod for me, can't play without it.
Since I'm a complete noob when it comes to coding, I just have one more question
Is it possible to implement  a config file for building upgrade coasts, like you did for update distance and spread factor?
I saw that a lot of people are complaining that buildings are leveling up too fast, but I actually liked that.

I have already promoted the modded building upgrade cost in 1.5.0. But yes, making building upgrade cost configurable is an option.

I have already promoted the modded building upgrade cost in 1.5.0. But yes, making building upgrade cost configurable is an option.

That would be great, I really liked it before, much quicker upgrade.
Thank you for your help, and for creating this mod. Looking forward for the future updates.

Same issue - low residential is still saying high rent - and abandoning.
Are these the expected startup mods?  It looks like it's on though.  But I don't see anything about changing land value in the logs.
I've cleared out everything in appData except save games.
I've put LandValueOverhaul mod first in the load order.

Same issue - low residential is still saying high rent - and abandoning.
Are these the expected startup mods? It looks like it's on though. But I don't see anything about changing land value in the logs.
I've cleared out everything in appData except save games. I've put LandValueOverhaul mod first in the load order.

There should be something like "Building upkeep modified!". In case your log does not contain this line, LVO is not correctly loaded, causing unusually high building upkeep, leading to high rent.

Update 1.5.1 fixed the info view system. The building upgrade cost remains not configurable for now, because the building upgrade cost should be something properly configured before starting a new save and remains static throughout the entire game.


ModSec regex not matching

Hi, we wanted to add an additional uri into our regex for one of our rule id's
SecRule REQUEST_URI "@rx /json/docs|/json/contact/([0-9]+)|/json/abc"
"id:9989,phase:2,t:none,t:lowercase,nolog,pass,ctl:ruleRemoveTargetByID=932150;ARGS"
we tried with a regex as above, with the specific ID, with the uri minus the regex part, tried changing it from by id to by tag and tried just having the uri for the contact part on its own. None of these worked for the /json/contact uri. We ended up having to whitelist the whole id which then worked
SecRuleRemoveById 932150
Any suggestions on what we could try so we arent removing by id?
Thanks
Scott

Hey Scott, I'm closing this as it's not a CRS issue, but rather a support request.
But now to your problem: The first things you should do is making sure your rule 9989 is placed before the CRS include. Then you could shift it from nolog to log o see if it triggers.


Fixed issue with panics when decoding frame header

Fixed an issue when decoding frames from a three octet slice.

I think I had this idea that sequence number suppression could be supported (a 802.15.4-2015 feature). But that is not something I need or have implemented.

Thanks! Merging.


Can't read *.parquet with new parquet code

my code
df = dd.read_parquet(
    's3://' + bucket_name + '/*.parquet',
    engine='fastparquet',
    gather_statistics=False
)
df.head()

error i got
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-24-03f5a773c126> in <module>
      2     's3://' + my_bucket_path + '/*.parquet',
      3     engine='fastparquet',
----> 4     gather_statistics=False
      5 )
      6 df.head()

/mnt/miniconda/envs/jestr-etl/lib/python3.7/site-packages/dask/dataframe/io/parquet/core.py in read_parquet(path, columns, filters, categories, index, storage_options, engine, gather_statistics, **kwargs)
    153         gather_statistics=gather_statistics,
    154         filters=filters,
--> 155         **kwargs
    156     )
    157     if meta.index.name is not None:

/mnt/miniconda/envs/jestr-etl/lib/python3.7/site-packages/dask/dataframe/io/parquet/fastparquet.py in read_metadata(fs, paths, categories, index, gather_statistics, filters, **kwargs)
     98                 pf.file_scheme = scheme
     99                 pf.cats = _paths_to_cats(fns, scheme)
--> 100                 relpath = paths.replace(base, "").lstrip("/")
    101                 for rg in pf.row_groups:
    102                     rg.cats = pf.cats

AttributeError: 'list' object has no attribute 'replace'

paths is the correct list of all my paths referenced by  's3://' + my_bucket_path + '/*.parquet'

cc @rjzamora

This looks like a bug for the case where the fastparquet engine is given a list of files, and gather_statistics=False - I'll fix the relpath definition.  Thanks for raising this issue @birdsarah!

@martindurant @mrocklin - This issue exposes a slight limitation with the current core-engine implementation.  The problem is in this untested block of code, which was (incorrectly) borrowed from the original design.
In the case that gather_statistics=False, we choose not to call pf = ParquetFile(...).  However, without scanning the metadata for every file, we cannot populate a list of row-group information to ultimately pass along to read_partition.
It looks like we need to allow the parts output of read_metadata to specify file-path information (rather than a row-group information) in the case that gather_statistics=False.  From there, read_partition can just read a specific file, rather than a specific row group.

closed by #5157


Marathon lost connection to zookeeper, now entire cluster refuses to start

Marathon lost its connection to ZK (I've been having noisy neighbor issues lately), and when it stopped it seems to have gone into a bad state.
Logs - https://gist.github.com/nemothekid/e13fe389fee9bf04c1b6
Whats strange here is the repeated
Nov  4 05:26:36 master-01 marathon[26260]: [2015-11-04 05:26:36,638] INFO Defeat leadership (mesosphere.marathon.MarathonSchedulerService$$EnhancerByGuice$$58a92378:280)
Nov  4 05:26:36 master-01 marathon[26260]: [2015-11-04 05:26:36,638] INFO Now standing by. Closing existing handles and rejecting new. (mesosphere.marathon.event.http.HttpEventStreamActor:96)
Nov  4 05:26:36 master-01 marathon[26260]: [2015-11-04 05:26:36,639] INFO Stopping driver (mesosphere.marathon.MarathonSchedulerService$$EnhancerByGuice$$58a92378:224)
Nov  4 05:26:36 master-01 marathon[26260]: [2015-11-04 05:26:36,639] INFO Defeated (Leader Interface) (mesosphere.marathon.MarathonSchedulerService$$EnhancerByGuice$$58a92378:235)

Almost 1000 times a second, then
Nov  4 05:26:37 master-01 marathon[26260]: [2015-11-04 05:26:37,065] ERROR Current member ID member_0000001157 is not a candidate for leader, current voting: [member_0000001282, member_0000001281] (com.twitter.common.zookeeper.CandidateImpl:144)
Nov  4 05:26:37 master-01 marathon[26260]: [2015-11-04 05:26:37,066] ERROR Current member ID member_0000001147 is not a candidate for leader, current voting: [member_0000001282, member_0000001281] (com.twitter.common.zookeeper.CandidateImpl:144)
Nov  4 05:26:37 master-01 marathon[26260]: [2015-11-04 05:26:37,067] ERROR Current member ID member_0000001189 is not a candidate for leader, current voting: [member_0000001282, member_0000001281] (com.twitter.common.zookeeper.CandidateImpl:144)
Nov  4 05:26:37 master-01 marathon[26260]: [2015-11-04 05:26:37,067] ERROR Current member ID member_0000001195 is not a candidate for leader, current voting: [member_0000001282, member_0000001281] (com.twitter.common.zookeeper.CandidateImpl:144)
Nov  4 05:26:37 master-01 marathon[26260]: [2015-11-04 05:26:37,068] ERROR Current member ID member_0000001150 is not a candidate for leader, current voting: [member_0000001282, member_0000001281] (com.twitter.common.zookeeper.CandidateImpl:144)

Now marathon cannot determine the current leader, restarting the cluster doesn't help.

Couldn't figure out whether this was a bug in marathon's leader election, or a problem in my cluster. Clearing out the leader state in ZK fixed the issue.


Add ng2-trim-directive to NPM

For quick install, can you add your project to npm?

I'm going to do that at this weekend, no question. 👍

Cool thanks man


Add history_graph component to demo

Description:
Add history graph to the demo.
Helps with UI development.
Example entry for configuration.yaml (if applicable):
demo:

Checklist:

[x] The code change is tested and works locally.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

CC @andrey-git


User interaction extension to add some advanced commands?

Now the user interaction extension supports telegram, discord, and email, but an interaction can only be bound to one agent, and agents cannot be switched during use. For convenience, it is best to add some commands that can interact with Opendan through association, such as switching associations. agent, new associated instances, etc.

You are very welcome to add some new commands.
After installing our plan, OpenDAN Desktop will simplify a large number of basic configurations for ordinary users through a graphical interface. I think the traditional command line is more to help developers.

What I’m talking about are commands for operating in telegram and discord, not traditional command line.
For example, the following commands are supported in telegram:
/list: List all currently supported agents
/set_default $target: switch default agent
/default: displays the current agent
@target: Send a message to the specified agent


Changed kv.bulk_io_write.max_rate to correct value

kv.bulk_io_write.max_rate was shown as 8 EiB but when I show the setting on a newly installed cluster the value is 1 TiB.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/635e5d471ac18a92f964e1152d7ab22f3eaa6f96/

As discussed on Slack, handling this in a separate PR with broader updates to our auto-generated docs for 19.2.

This change is


feat: implement predicate_query to validate which are the metrics to be collected

What
Implement a validation logic to verify if a query to collect metrics should be executed or not.

ref: #4499

How

Define a property predicate_query.
Define a method isCollectable that is invoked by collect passing the same tx (to reduce the performance consumption) that returns a bool.
Add a new e2e test, with related files, to check the scenarios of predicate_query.

Test

Execute the e2e-test-local with FEATURE_TYPE= observability and TEST_DEPTH=3.

Open points - Questions

Do you think there is a need to add any specific log when a query is not executed due to the false response of the predicate_query? I want to avoid bloating PG with this kind of log so this is why currently it is not present.
isCollectable is intended to be invoked only within the collect method, maybe I could define it as anonymous method inside the collect?
We can simplify a lot the logic in the isCollectable method if we assume strong and tight rules regarding the output of the predicate_query, what do you think about that?
Do I have to update the doc within this PR or, when approved, I will create another PR to document what is and it is working the predicate_query?

Thanks at disposal for any question 🙇 🚀

Do you think there is a need to add any specific log when a query is not executed due to the false response of the predicate_query? I want to avoid bloating PG with this kind of log so this is why currently it is not present.

No, it definitely should not log, or not at above debug level. That's part of the point, so unwanted log spam can be avoided when the target server doesn't support a specific view, column, function etc.
In general there is IMO a need to improve how CNP's metrics scraper diagnostics work and make it easier to understand its behaviour + detect errors. That's why I wrote a little tool to run a one-shot scrape against a pre-configured postgres recently. I'll see if I can tidy that up for a community contribution. But logging isn't IMO the answer to that. I already have enough problems with it quietly logging a complaint and happily continuing with some confusing behaviour.
It might make sense to emit metrics as scrape output to report on skips, but they should probably be optional, and I don't personally think they're necessary as part of the basic feature. They could easily bloat scrape results and add unwanted scrape cardinality, requiring more filtering by scrape result consumers to discard the expected ones.

We can simplify a lot the logic in the isCollectable method if we assume strong and tight rules regarding the output of the predicate_query, what do you think about that?

The problem with that is that the user has to reliably assume the query has the expected results under all circumstances. Part of the point of this is to make the CNP scraper robust in the face of differing server versions, extension installs, vendor flavours, etc etc.
OTOH if the query does something unexpected, a clear error log might be better than just silently returning false and not running the query. IDK. Right now it's hard to properly test CNP scraper configs across a wide matrix of possible server configs, so I prefer to be error-tolerant, but I can see the argument for strictness too.
I'd personally prefer it to accept a 0-row result as false, but it's not hard to wrap that in a SELECT EXISTS (...) if you want to require a strict boolean result.

/test tl=4 l=local

@mnencia What can we do to progress this PR?

@gabrosys I see a test failure

[FAIL] Metrics [It] can gather metrics according with predicate query [observability]
/home/runner/work/cloudnative-pg/cloudnative-pg/tests/e2e/asserts_test.go:2926

https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/9131921366/job/25112487661#step:9:5716
It failed in some of the other runs too, but passed in others.
Details https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/9131921366/job/25112490237#step:9:2344
   Timeline >>
  Putting Tail on the operator log
  STEP: verifying the custom metrics ConfigMaps and Secrets exist @ 05/17/24 18:30:22.086
  STEP: setting up curl client pod @ 05/17/24 18:30:22.245
  STEP: having a predicate-query-metrics-e2e-9171 namespace @ 05/17/24 18:30:27.285
  STEP: creating a Cluster in the predicate-query-metrics-e2e-9171 namespace @ 05/17/24 18:30:27.291
  STEP: having a Cluster postgresql-metrics with each instance in status ready @ 05/17/24 18:30:27.335
  Cluster ready, took 1m12.298617472s
  STEP: ensuring metrics with positive predicate are collected @ 05/17/24 18:31:39.665
  STEP: checking metrics for pod: postgresql-metrics-1 @ 05/17/24 18:31:39.672
  [FAILED] in [It] - /home/runner/work/cloudnative-pg/cloudnative-pg/tests/e2e/asserts_test.go:2926 @ 05/17/24 18:31:40.062
  DUMPING tailed Operator Logs with error/warning (at most 10 lines ). Failed Spec: Metrics can gather metrics according with predicate query

  ================================================================================
  -- no error / warning logs --
  ================================================================================
  << Timeline

  [FAILED] Found no match for metric cnpg_pg_predicate_query_return_true_and_multiple_rows_fixed

  Priting rawMetricsOutput:
...
  cnpg_errors_total{errorUserQueries="pg_predicate_query_return_true_and_multiple_rows on db app: ERROR: subquery in FROM must have an alias (SQLSTATE 42601)"} 1
...
  # HELP cnpg_pg_predicate_query_return_true_and_multiple_columns_fixed Always 42, used to test predicate_query
  # TYPE cnpg_pg_predicate_query_return_true_and_multiple_columns_fixed gauge
  cnpg_pg_predicate_query_return_true_and_multiple_columns_fixed 42
  # HELP cnpg_pg_predicate_query_return_true_and_multiple_columns_multiple_rows_fixed Always 42, used to test predicate_query
  # TYPE cnpg_pg_predicate_query_return_true_and_multiple_columns_multiple_rows_fixed gauge
  cnpg_pg_predicate_query_return_true_and_multiple_columns_multiple_rows_fixed 42
  # HELP cnpg_pg_predicate_query_return_true_fixed Always 42, used to test predicate_query
  # TYPE cnpg_pg_predicate_query_return_true_fixed gauge
  cnpg_pg_predicate_query_return_true_fixed 42

so it looks like an issue with the spec for pg_predicate_query_return_true_and_multiple_rows

Update the PR as requested. Now we accept only predicate_query with at most one row and with a single bool column.
In detail, we handle:

true and false
no row as false
null as false

@jsilvela ready to be reviewed, thanks 🙇

/test limit=local

/test limit=local

Hello @gabrosys could you please integrate the User defined metrics doc section with the new field and explain how to use it?

Update the doc with a new paragraph with a simple example and add the new prop to the prop list. Thank you all for your support 🙇


chore(deps): update renovatebot/github-action action to v34.138.3

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
minor
v34.136.0 -> v34.138.3

Release Notes

renovatebot/github-action
v34.138.3
Compare Source
See the the changelog for changes in all releases.
34.138.3 (2023-02-15)
Bug Fixes

deps: update renovate/renovate docker tag to v34.138.3 (600bc8a)

v34.138.1
Compare Source
See the the changelog for changes in all releases.
34.138.1 (2023-02-14)
Bug Fixes

deps: update renovate/renovate docker tag to v34.138.1 (7183fed)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add outdated alias

A detailed description of the proposed feature
Is it possible to add out alias to outdated?

The motivation for the feature
To be more convenient. npm has one.

How the feature would be relevant to at least 90% of Homebrew users
Will be useful I think.

What alternatives to the feature have been considered
Typing outdated all the time.

The relevant code is here:

https://github.com/Homebrew/brew/blob/master/Library/Homebrew/brew.sh#L269-L286

Use https://github.com/Homebrew/homebrew-aliases or a Bash alias.


If the discovery service is down calls will fail even with caching enabled

Create a custom discovery service that extends CachingServiceDiscovery, integrating with an external service (e.g. consul).
Define a refresh period of 10 seconds or so.
Start the client application and server application and get the client to make a few calls on the server application (it will start by fetching the server url from the external discovery service).
Now shut this external discovery service down, and try to get the client to continue calling the server app.
After the refresh period, an attempt will be made to contact the discovery service, which will fail. As a result all business calls will fail, until the discovery service is restored.
The access to the discovery service should be more resilient (although the discovery service should be very resilient in itself, since it is a central and critical service), especially when services have been successfully fetched at least once.
The proposed enhancement would be to return the previously fetched instances (if any) if fetching new instances fail.
There should be also a configurable read timeout and connect timeout on the discovery. Since getting a valid instance is probably blocking for the client, we do not want to make him wait indefinitely, or even on a large connect or read timeout (e.g. 30 secs), if some instances have been fetched in the past.
Refreshing the instances from the discovery service should be seen as "best effort", when instances have been previously fetched in the past.
On the very first attempt, when there are not cached instances, we may use a bigger read and connect timeouts, because if we fail fetching instances, the business call will fail anyway. So it is worth waiting a little bit longer.
But when instances exist and have been cached, we should wait for a very short period of time only (e.g. 100ms depending on the 90% response time of the external discovery service), and fall back quickly on the cached instances if we do not get an answer in time.

hello @michalszynkiewicz
I have created a branch for that specific issue: https://github.com/vsevel/stork-issue/tree/issue_231
Instructions to reproduce:
mvn clean install

# first window (app)
java -Dquarkus.http.port=8080 -jar target/quarkus-app/quarkus-run.jar

# second window (discovery service)
java -Dquarkus.http.port=8060 -jar target/quarkus-app/quarkus-run.jar

# third window
 curl http://localhost:8080/hello/stork => SUCCESS
 curl http://localhost:8080/hello/stork => SUCCESS
...

now kill discovery service and wait refresh period
 curl http://localhost:8080/hello/stork => KO
 curl http://localhost:8080/hello/stork => KO
...

restart discovery service and wait refresh period or so:
 curl http://localhost:8080/hello/stork => SUCCESS
 curl http://localhost:8080/hello/stork => SUCCESS
...

Does your service discovery return a Uni with a failure or throw an exception?

it is returning a Uni:
    @GET
    @Produces(MediaType.TEXT_PLAIN)
    public Uni<String> discovery(@QueryParam("app") String app) {
        String address = addresses.get(app);
        UniCreate uni = Uni.createFrom();
        if (address != null) {
            log.info("discovery returning " + address + " for app " + app);
            return uni.item(address);
        } else {
            return uni.failure(new RuntimeException("could not find address for app " + app));
        }
    }

did I make any mistake?

how do you invoke it from the ServiceDiscovery implementation?

    @Override
    public Uni<List<ServiceInstance>> fetchNewServiceInstances(List<ServiceInstance> previousInstances) {
        DiscoveryClient discoveryClient = Arc.container().instance(DiscoveryClientProvider.class).get().getClient();
        long timeout = previousInstances.isEmpty() ? 10000L : 100L;
        String addresses = discoveryClient.discovery(config.getApp()).await().atMost(Duration.ofMillis(timeout));
        UniCreate uni = Uni.createFrom();
        try {
            return uni.item(map(addresses));
        } catch (TimeoutException e) {
            return uni.failure(e);
        }
    }

I need to rebase on your code that you provided in the PR, to see if that changes anything.

note also that I did not see that behavior in 1.0.0 with <artifactId>stork-load-balancer-response-time</artifactId>
If I did not mixed up my tests, it seems a (negative) change of behavior in 1.0.1-SN

hmm. not seeing the issue anymore after the rebase on my project (including your code).
still surprising to me because even if was doing some blocking code, I was catching the mutiny timeout, and resending a uni failure.
still, I do not see a reason to keep that issue opened, unless you need me to make additional tests in that context.
I am seeing a log error in the CachingServiceDiscovery with a long stacktrace when the discovery service is not reachable:
2022-02-10 12:41:38,318 ERROR [io.sma.sto.imp.CachingServiceDiscovery] (vert.x-eventloop-thread-1) Failed to fetch service instances: javax.ws.rs.ProcessingException: io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: localhost/<IP_ADDRESS>:8060
	at org.jboss.resteasy.reactive.client.handlers.ClientSendRequestHandler$2.accept(ClientSendRequestHandler.java:230)
	at org.jboss.resteasy.reactive.client.handlers.ClientSendRequestHandler$2.accept(ClientSendRequestHandler.java:226)
	at io.smallrye.context.impl.wrappers.SlowContextualConsumer.accept(SlowContextualConsumer.java:21)
	at io.smallrye.mutiny.helpers.UniCallbackSubscriber.onFailure(UniCallbackSubscriber.java:65)
	at io.smallrye.mutiny.operators.uni.UniOperatorProcessor.onFailure(UniOperatorProcessor.java:54)

Since the business call works regardless using the cache, an option would be to log a warn only (with no stack), or in debug with a stack to cut on verbosity.

for sake of completion I have seen this warning while I was running my test (on the app side):
2022-02-10 12:45:56,569 WARN  [io.ver.cor.imp.VertxImpl] (executor-thread-0) You're already on a Vert.x context, are you sure you want to create a new Vertx instance?
2022-02-10 12:46:40,715 WARN  [io.ver.cor.imp.VertxImpl] (executor-thread-0) You're already on a Vert.x context, are you sure you want to create a new Vertx instance?

It happened only twice (but the discovering service was called multiple times).

oh, it's a bug in Quarkus, I'll try to fix it. Created an issue for it https://github.com/quarkusio/quarkus/issues/23578

as the last thing is not related to the report, I'll close this issue. If I missed something please reopen it


Heroku CLI tar download URL

It looks like that the current Heroku CLI features downloads an older version of the Heroku CLI, even if the version argument is latest
https://github.com/devcontainers-contrib/features/blob/9a1d24b27b2d1ea8916ebe49c9ce674375dced27/src/heroku-cli/install.sh#L53
It will download 7.69.1 version, instead of 8.11.5 (as of today)
That's probably because Heroku tar files position is changed.
For example, if you go here: https://devcenter.heroku.com/articles/heroku-cli#tarballs you'll see that the Linux x64 tar file is located here:
https://cli-assets.heroku.com/channels/stable/heroku-linux-x64.tar.gz
Instead, this feature will try to download it from here:
https://cli-assets.heroku.com/heroku-linux-x64.tar.gz
The latter is still a valid URL, but for an old version of the Heroku CLI

The problem of having an older version of Heroku CLI is that it raises an error on my container
root ➜ /app (master) $ heroku
qemu-arm: Could not open '/lib/ld-linux-armhf.so.3': No such file or directory

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


Blazor SPA PWA offline auth

[EDIT by guardrex to add the metadata. @Eweol ... Please in the future use the This page feedback button and form at the bottom of the topic to open an issue here.]
Hey guys,
I had a look hole 2 days to find a solution for my problem regarding offline authentication of a user in my Blazor Standalone PWA.
With .Net 5.0 I changed the Authentication scenario from Web to SPA.
After that change, my PWA was not able to login the user in offline mode.
I work on Program.cs with AddMsalAuthentication and save the tokens and ID's in local storage with help of MSAL.
If I'm online everything works like expected and I can login with the users and the tokens will be stored in local storage.
After changing the online state the PWA try to get the openid-configuration data from azure.
I tried to also serve this request with the service worker, which worked quiet fine but the token request after that fail also.
I searched and found the CarChecker app and the docs as well but this didn't solve my problem.
Did I missed something or is there really no way, to make offline auth workable on Blazor standalone PWA's?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7c8c235e-1ce4-046f-e848-294096361922
Version Independent ID: 6240f626-97d4-ace0-5e34-20e1c3592f86
Content: Build Progressive Web Applications with ASP.NET Core Blazor WebAssembly
Content Source: aspnetcore/blazor/progressive-web-app.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

OOF for the :gift: holidays :snowman:
This issue has been marked for triage on the Blazor.Docs project, and I'll respond as soon as I return in 2021.
We only work on documentation on this repo. If you need product support, close this issue and seek assistance through one or more of the following support channels:

Stack Overflow (tagged: blazor)
General ASP.NET Core Slack Team
Blazor Gitter

If you think that you found a potential bug in the framework or have product feedback, close this issue and open a new issue for the ASP.NET Core product unit at aspnet/AspNetCore issues.
For problems or feedback on Visual Studio or Visual Studio for Mac, close this issue and use the Report a Problem or Suggest a Feature processes from within VS, which open internal issues for the VS team. For more information, see Visual Studio feedback options or How to report a problem in Visual Studio for Mac.
For problems with Visual Studio Code, close this issue and ask for support on community support forums. For bug reports and product feedback, open an issue on the microsoft/vscode GitHub repo.
⛄🎁 Happy Holidays! :dancer::man_dancing::partying_face: Happy New Year! :balloon::champagne::tada:

OOF for the :gift: holidays :snowman:
This issue has been marked for triage on the Blazor.Docs project, and I'll respond as soon as I return in 2021.
We only work on documentation on this repo. If you need product support, close this issue and seek assistance through one or more of the following support channels:

Stack Overflow (tagged: blazor)
General ASP.NET Core Slack Team
Blazor Gitter

If you think that you found a potential bug in the framework or have product feedback, close this issue and open a new issue for the ASP.NET Core product unit at aspnet/AspNetCore issues.
For problems or feedback on Visual Studio or Visual Studio for Mac, close this issue and use the Report a Problem or Suggest a Feature processes from within VS, which open internal issues for the VS team. For more information, see Visual Studio feedback options or How to report a problem in Visual Studio for Mac.
For problems with Visual Studio Code, close this issue and ask for support on community support forums. For bug reports and product feedback, open an issue on the microsoft/vscode GitHub repo.
⛄🎁 Happy Holidays! :dancer::man_dancing::partying_face: Happy New Year! :balloon::champagne::tada:

I'm back from vacation.
I haven't received any additional info from the product unit outside of what we have published at ...
https://docs.microsoft.com/en-us/aspnet/core/blazor/progressive-web-app?view=aspnetcore-5.0&tabs=visual-studio#interaction-with-authentication
There is an open issue that I'll follow-up on at ...
https://github.com/dotnet/AspNetCore.Docs/issues/20742
... that pertains to PWAs and auth. I don't think that it's the same problem, but you should look that over (and the product unit issue on it) to confirm that it's not related/helpful for your scenario.
I think if you're hitting problems with basic auth outside of the minimal doc coverage and in CarChecker, you have two courses of action that you can take. If you think the community can help out because others have developed PWA-auth apps successfully, especially CarChecker, you can hit up the community on the usual support channels ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

Second, you can ask the product unit directly by opening an issue on their repo with the problems/errors that you're seeing ...
https://github.com/dotnet/aspnetcore/issues
Of course, they'll need specifics ... error messages ... behaviors ... versions of everything in use ... and perhaps a small repro project that demonstrates the problem(s).
What I'll do for that doc section is give it my own go when I reach this topic for it's UE ("user experience") pass, which is when I'll be testing everything in the doc and updating/overhauling it, on https://github.com/dotnet/AspNetCore.Docs/issues/19286. I'll mark that tracking issue with a cross-link to this issue to remind me about your issue.
If you do open an issue for the product unit, please put a cc: @guardrex at the bottom of your opening comment so that I can follow the discussion.

I'm back from vacation.
I haven't received any additional info from the product unit outside of what we have published at ...
https://docs.microsoft.com/en-us/aspnet/core/blazor/progressive-web-app?view=aspnetcore-5.0&tabs=visual-studio#interaction-with-authentication
There is an open issue that I'll follow-up on at ...
https://github.com/dotnet/AspNetCore.Docs/issues/20742
... that pertains to PWAs and auth. I don't think that it's the same problem, but you should look that over (and the product unit issue on it) to confirm that it's not related/helpful for your scenario.
I think if you're hitting problems with basic auth outside of the minimal doc coverage and in CarChecker, you have two courses of action that you can take. If you think the community can help out because others have developed PWA-auth apps successfully, especially CarChecker, you can hit up the community on the usual support channels ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

Second, you can ask the product unit directly by opening an issue on their repo with the problems/errors that you're seeing ...
https://github.com/dotnet/aspnetcore/issues
Of course, they'll need specifics ... error messages ... behaviors ... versions of everything in use ... and perhaps a small repro project that demonstrates the problem(s).
What I'll do for that doc section is give it my own go when I reach this topic for it's UE ("user experience") pass, which is when I'll be testing everything in the doc and updating/overhauling it, on https://github.com/dotnet/AspNetCore.Docs/issues/19286. I'll mark that tracking issue with a cross-link to this issue to remind me about your issue.
If you do open an issue for the product unit, please put a cc: @guardrex at the bottom of your opening comment so that I can follow the discussion.


Review timeout arguments

They should be int? timeout = null
https://github.com/microsoft/playwright-sharp/pull/1113/files#diff-a8d5f823294cea4f75ff4d733c99b11f61e5ce4aa4bd98b186da9adc83141101R131

Would it be more intuative to have Timespan? instead of int? for timeout args? I keep forgetting if it's seconds or milliseconds when it's just int? timeout = null.

What do you think about that @avodovnik ?

We've talked a lot about this, if you remember @kblok. Especially in the context of using language specific types, like Timespan and FileInfo, etc. That said, I do think, genuinely, Timespan would be a very handy one here.
When we're nearly done with the alpha, let's see if we can squeeze this one in, how's that? I want to make sure we don't introduce additional (major) breaking changes after the alpha.


Offline changes lost?

First of all, thanks for your great work. However I am wondering the following:
If I make changes (e.g. data creations, updates and deletions) offline, will they ever be uploaded/ synchronized to/ with the online remote database? Because by looking at the code it seems to me that your plugin only allows accessing data offline that was previously retrieved by the online database and whenever we go online again, the plugin updates the local (offline) database using the online database.
I would be very happy about your help. :)

Would love to accept PRs!


Add support for ReadOf behavior

As titled! 🚀
This will solve most of the issues identified in #544.
cc @droberts195

I’m not sure if ReadOf is the best name, we can generalize it further and call it CopyOf or SameAs or something along those lines. What do you think?


⚠ Bofrost.lu has degraded performance

In 2e55fd3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2847 ms

Resolved: Bofrost.lu performance has improved in 751901e after 18 minutes.


python310Packages.coincurve: 17.0.0 -> 18.0.0

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

NACK. We should fix Electrum first.

NACK
This breaks Electrum-GRS, we need to address this before this PR gets merged.

Regarding Electrum and variants, it seems like the problem is that the secp256k1 derivation now calls its library libsecp256k1.so.1.0.0 instead of libsecp256k1.so.0, which is what Electrum expects. A non-Nix installation of Electrum with libsecp256k1 0.2 does not have this problem, so it looks like a bug on our side.

Are there any open PRs I can review and merge to fix this?

Are there any open PRs I can review and merge to fix this?

I don't think so.

Otherwise I would mark them broken due to lack of time on my side.

If you don't have time for this, we should not update the package that does the breakage.

Above said, I will have a look at the issue tomorrow and fix Electrum. The bug is inside Nixpkgs after updating secp256k1, not in the Electrum upstream.

I dug a bit deeper. It looks like https://github.com/bitcoin-core/secp256k1/commit/e025ccdf7473702a76bb13d763dc096548ffefba changed the ABI version on purpose. It's not a bug on our side.
libbitcoin has its own fork of libsecp256k1 so their package could use that. Or we could patch it.
Regarding Electrum and derivatives, they're missing the recover module.

@SuperSandro2000 can you rebase? Otherwise I can take over.

rebased and cleaned up

Superseded by #208871 and #208232

and #214212 and #214213


Enable 4.7 cluster on periodic jobs

/kind user-story
/area testing
User Story
As a user I want to run periodic jobs on 4.7 cluster
Acceptance Criteria

[ ] Fixing the blockers (if any) for enabling 4.7 cluster on Periodic jobs
- https://github.com/openshift/odo/issues/4303
[ ] Jobs should run successfully on 4.7 ocp cluster.

Links

Feature Request: NA
Related issue: NA

/kind user-story

It s blocked on https://github.com/openshift/odo/issues/4303

[ ]  Fixing the blockers (if any) for enabling 4.7 cluster on Periodic jobs

Can you please link blocker issues here?

Fixed via https://github.com/openshift/release/pull/14234

Fixed via https://github.com/openshift/release/pull/14234


dts resolve to CJS entry dts instead of ESM entry dts when not included in tsconfig

Bug Report

dts resolve to CJS entry dts instead of ESM entry dts when js not included in tsconfig
🔎 Search Terms

dts resolve priority
tsconfig
javascript
// @ts-check

🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link

Playground link with relevant code
💻 Code

await import('@sinclair/typebox')

https://github.com/loynoir/reproduce-typescript-53958
$ npm exec -- tsc --showConfig  | grep checkJs
        "checkJs": true
$ npm exec -- tsc --showConfig  | grep include -A 4
    "include": [
        "./src/**/*",
        "./test/**/*",
        "./reproduce2/**/*"
    ],

🙁 Actual behavior

./reproduce1/test-esm-dts-resolve-1.mjs
./reproduce1/test-esm-dts-resolve-2.mjs
./reproduce1/test-mts-dts-resolve.mts

dts resolve to<EMAIL_ADDRESS>🙂 Expected behavior

dts resolve to<EMAIL_ADDRESS>

The files in reproduce1 don't correspond to any tsconfig file, so nothing causes them to have moduleResolution: "nodenext" as is required to do node16 module resolution


Crowd linux

Changes proposed ✍
What
Changed the links on the 403 error page to Linux Foundation specific.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


pylon is not ready yet. Please wait for a moment!

启动pai服务时，卡在这里，images已下载好了
管理界面的日志只有一句话
container "pylon" in pod "pylon-ds-hhkqx" is waiting to start: ContainerCreating
请问在哪里看卡住的详细原因

我也卡在这，但是问题是Failed create pod sandbox.

之前找到过一些原因，是因为docker镜像下载慢，还有一个是端口被用了，换了个端口就好了，但是偶尔还是会卡住，找不到哪里能详细原因

Hi, you can check the methods in this issue.
The common reason for pylon not ready is port conflict or firewall.
You can also check kubelet log in your dev box by docker logs kubelet.

thx, i solve this problem by uncommenting pylon port:88 in services-configuaration.yaml , it confilict with my nginx.


Add Filter Application using Flask and OpenCV.

Is your feature request related to a problem? Please describe.
In this project, I have used Flask and OpenCV to add filters to the frames of the video.
This project combines the power of Flask and OpenCV to create a user-friendly application where users can apply various filters to video frames. It provides an interactive and intuitive interface for experimenting with different filters and enhancing the visual appeal of images.
Describe the solution you'd like
The UI shows up the video and several filter options which can be applied to the video frame by the user.
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I have completed the project as you can check HERE . Once assigned to me I shall come up with a PR.

@lakshmishreea122003 - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.


Translating ServiceLocator, Repository and Delegation patterns to pt_BR

This PR includes translations for Service Locator, Repository and Delegation patterns to brazilian portuguese.

Merged, thank you @leonampd!


Updates

[x] Change default app prefix to +90 instead of +91. 1.1.92-beta

[x] In the main app page, a space is required between "Oturum" and blue "aç". 1.1.92-beta

[ ] I added my phone number to Doctors Database collection. When I select Doctor on the sign in page, it says Approved however a blank screen comes afterwards. Do I need to add somewhere else? 1.1.92-beta

You do not need to add it anywhere else. Sometimes, because of internet issues (very rarely), the app might have problems connecting to Firebase. Just restart the app and it should work fine.

You do not need to add it anywhere else. Sometimes, because of internet issues (very rarely), the app might have problems connecting to Firebase. Just restart the app and it should work fine.

The issue in this particular case was because a string was being passed as null to a Widget. This happened because of the Turkish Translation. We need to check for more such possible cases before release.


fix bug about PUBCOMP

Hi Tuan,
Thanks for your mqtt component, it's very helpful for me.
When I am using it, I think I found a little bug, which causes the device can't finish QoS2 publish.
I  fixed it and now open this PR. Hope it's the right way.
Best Regards,

I do not really check qos2, thanks


Doesn't install

I have installed all the dependencies for electron-acrylic-window (windows-build-tools) and it won't install when I run  npm i electron-acrylic-window --save. Here are the logs:
2021-01-10T15_20_26_542Z-debug.log
EDIT: Ran the command in CMD instead of PowerShell, different errors.
2021-01-10T15_30_13_035Z-debug.log

Duplicate of #46. Please use version 0.4.7 before the modifications are applied.

Duplicate of #46. Please use version 0.4.7 before the modifications are applied.

@meisme-dev It has resolved on v0.5.5!

@meisme-dev It has resolved on v0.5.5!


7 update portal plugin to reflect schema changes

See #7 for more details.

Adding @BenjaminDewey as a watcher.


22845 -  Update Confirm Page Delivery Email

Issue #: /bcgov/entity#22845
Description of changes:

For the review & confirm page, updated the Documents Delivery Section:
Made the Completing Party email address editable
Keep prepopulating the default staff email address
Only when it's a staff filing
Updated for Amalgamation, Dissolution, Continuation, Incorporation, Registration.
Restoration seems not needed/ not updated.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-create-dev--pr-729-0xdxe2m1.web.app

Dissolution before update:

Keep the same design, update the default staff email address:

For Amalgamation, Dissolution, Continuation, Incorporation, and Registration, use the same design as the above:

When it's not a staff filing, keep previous design, not changed:

Restoration seems not needed/ not updated:

The 2 emails are from step 2 and step 3

/gcbrun

Temporary Url for review: https://business-create-dev--pr-729-0xdxe2m1.web.app

Tested with:
Amalgamation

Staff Filing
https://dev.create.business.bcregistry.gov.bc.ca/amalg-reg-review-confirm?id=TL1Eo8Ugjz&accountid=3040
Regular User (not changed)
https://dev.create.business.bcregistry.gov.bc.ca/amalg-reg-review-confirm?id=T4wvaHoU09&accountid=3302

Dissolution  - Staff Filing
https://dev.create.business.bcregistry.gov.bc.ca/dissolution-review-confirm?id=C9900784&accountid=3040
Continuation

Staff Filing
https://dev.create.business.bcregistry.gov.bc.ca/continuation-in-review-confirm?id=TCzVSqSar4&accountid=3040
Regular User (not changed)
https://dev.create.business.bcregistry.gov.bc.ca/continuation-in-review-confirm?id=ThevF8a4zF&accountid=3302

Incorporation - Staff Filing
https://dev.create.business.bcregistry.gov.bc.ca/incorporation-review-confirm?id=T5lYTaTKSu&accountid=3040
Registration - Staff Filing
https://dev.create.business.bcregistry.gov.bc.ca/registration-review-confirm?id=TM1hgJDxOp&accountid=3040


Show version in output

Currently need to look at pip freeze to identify the current running version.  Would be great if all commands indicated current version.
Other alternative is to have banana version or banana -v.

will do, thanks

Hey, I tried solving this, I have added a pr as well https://github.com/bananaml/banana-cli/pull/22

This has been implemented as a --version flag in a recent update using the native click package decorator. 👍


Return overlays if user has rw permissions

This fixes #1632. @xavijam CR this, please.

Perfect. :+1:


Why do you only use the "maybe-require-package" on diminish in ivy settings?

Hello Purcell, Sorry for taking your time again.
I've been reading your code to avoid me raised these annoying little problems again.
But when I read these two places, I didn't understand again.,,
In your lisp/init-ivy.el
    (when (maybe-require-package 'diminish)
      (diminish 'ivy-mode)))

  (when (maybe-require-package 'diminish)
    (after-load 'counsel
      (diminish 'counsel-mode)))

Why do you only use "maybe-require-package" for diminish in these both places? I see you did not use this judgment function in other places. You just used (diminish 'xxx).
Will there it be in danger to don't use maybe-require-package here ?
Thank you.

I use maybe-require-package when I know that the package may not be compatible with a version of Emacs supported by this config. Ideally all of the config would use that rather than require-package, but I'm not ready to fix it all. use-package would be a good package to use if I did that.

Thanks.

In other words, the latest version can remove it?
And, in fact, I have always had a problem ...
Why DO NOT you use use-package?
Use-package seems to be used by many people to simplify their configuration ...
Because migration requires a great cost you?


Fixes to scRNA material

More fixes, updated details, and subsetted the preprocessing data due to massive bottleneck in processing (Reduction from 40 minutes to 5 minutes)

Good luck tomorrow!


Add option to download application version from app.json

Hi @sergiogp1, great project! Love the idea and it works like a charm!
One minor feature request that could be put in the backlog for now:
Given an app.json file, we could read the application version and provide an additional option to download the specific version or as waldo put it in https://github.com/sergiogp1/BC-AL-Packages/issues/3, provide the latest of the specific major-minor version.
app.json:
{
    ...
    "application": "21.1.48363.57447",
    ...
}

In this case it would provide an option to download the version:  21.1.48363.57447
If the application version is <IP_ADDRESS> than provide the option to download the latest version for 21.1.x.x
If the application version is <IP_ADDRESS> than provide the option to download the latest version of 22.x.x.x
What do you think? Thanks in advance

I like your idea :)
So, maybe one possibility is that when you are asked to choose between Sandbox or OnPrem, add a third option that would download symbols directly based on your on app.json, right?

Yes, so three options in total.
Also thinking about making it an extension setting, but that would be a bit too much for now. We would exclude the Sandbox/OnPrem question by making our app.json the source of truth.