Rate Limit Exceeded

Version of the custom_component
0.2.2
Configuration
Poll interval = 10
Describe the bug
In the HA log file, I'm getting frequent repeated error messages every 10s (see below) saying that the govee api rate limit has been exceeded.
I have 2 govee accounts setup, both with the same polling interval. One is working fine, but this 2nd one keeps coming up with this error.
Debug log
2023-03-02 09:49:21.460 WARNING (MainThread) [govee_api_laggat.api] Rate limit exceeded, check if other devices also utilize the govee API
2023-03-02 09:49:21.463 WARNING (MainThread) [govee_api_laggat.govee_api_laggat] error getting state for device GoveeDevice(device='8F:E4:D4:AD:FC:9E:51:22', model='H6182', device_name='TV LEDs', controllable=True, retrievable=True, support_cmds=['turn', 'brightness', 'color', 'colorTem'], support_turn=True, support_brightness=True, support_color=True, support_color_tem=True, online=True, power_state=False, brightness=254, color=(255, 255, 255), color_temp=0, timestamp=1677748855.467994, source=<GoveeSource.API: 'api'>, error='API-Error 429: Rate limit exceeded, retry in 19 seconds.', lock_set_until=0, lock_get_until=0, learned_set_brightness_max=100, learned_get_brightness_max=100, before_set_brightness_turn_on=False, config_offline_is_off=False): API-Error 429: Rate limit exceeded, retry in 11 seconds.
2023-03-02 09:49:21.465 WARNING (MainThread) [custom_components.govee.light] update failed for 8F:E4:D4:AD:FC:9E:51:22: API-Error 429: Rate limit exceeded, retry in 11 seconds.
2023-03-02 09:49:22.162 WARNING (MainThread) [govee_api_laggat.api] Rate limit exceeded, check if other devices also utilize the govee API
2023-03-02 09:49:22.164 WARNING (MainThread) [govee_api_laggat.govee_api_laggat] error getting state for device GoveeDevice(device='8F:E4:D4:AD:FC:9E:51:22', model='H6182', device_name='TV LEDs', controllable=True, retrievable=True, support_cmds=['turn', 'brightness', 'color', 'colorTem'], support_turn=True, support_brightness=True, support_color=True, support_color_tem=True, online=True, power_state=False, brightness=254, color=(255, 255, 255), color_temp=0, timestamp=1677748855.467994, source=<GoveeSource.API: 'api'>, error='API-Error 429: Rate limit exceeded, retry in 11 seconds.', lock_set_until=0, lock_get_until=0, learned_set_brightness_max=100, learned_get_brightness_max=100, before_set_brightness_turn_on=False, config_offline_is_off=False): API-Error 429: Rate limit exceeded, retry in 19 seconds.
2023-03-02 09:49:22.166 WARNING (MainThread) [custom_components.govee.light] update failed for 8F:E4:D4:AD:FC:9E:51:22: API-Error 429: Rate limit exceeded, retry in 19 seconds.
2023-03-02 09:49:31.499 WARNING (MainThread) [govee_api_laggat.api] Rate limit exceeded, check if other devices also utilize the govee API
2023-03-02 09:49:31.501 WARNING (MainThread) [govee_api_laggat.govee_api_laggat] error getting state for device GoveeDevice(device='8F:E4:D4:AD:FC:9E:51:22', model='H6182', device_name='TV LEDs', controllable=True, retrievable=True, support_cmds=['turn', 'brightness', 'color', 'colorTem'], support_turn=True, support_brightness=True, support_color=True, support_color_tem=True, online=True, power_state=False, brightness=254, color=(255, 255, 255), color_temp=0, timestamp=1677748855.467994, source=<GoveeSource.API: 'api'>, error='API-Error 429: Rate limit exceeded, retry in 19 seconds.', lock_set_until=0, lock_get_until=0, learned_set_brightness_max=100, learned_get_brightness_max=100, before_set_brightness_turn_on=False, config_offline_is_off=False): API-Error 429: Rate limit exceeded, retry in 11 seconds.
2023-03-02 09:49:31.503 WARNING (MainThread) [custom_components.govee.light] update failed for 8F:E4:D4:AD:FC:9E:51:22: API-Error 429: Rate limit exceeded, retry in 11 seconds.
2023-03-02 09:49:32.163 WARNING (MainThread) [govee_api_laggat.api] Rate limit exceeded, check if other devices also utilize the govee API
2023-03-02 09:49:32.166 WARNING (MainThread) [govee_api_laggat.govee_api_laggat] error getting state for device GoveeDevice(device='8F:E4:D4:AD:FC:9E:51:22', model='H6182', device_name='TV LEDs', controllable=True, retrievable=True, support_cmds=['turn', 'brightness', 'color', 'colorTem'], support_turn=True, support_brightness=True, support_color=True, support_color_tem=True, online=True, power_state=False, brightness=254, color=(255, 255, 255), color_temp=0, timestamp=1677748855.467994, source=<GoveeSource.API: 'api'>, error='API-Error 429: Rate limit exceeded, retry in 11 seconds.', lock_set_until=0, lock_get_until=0, learned_set_brightness_max=100, learned_get_brightness_max=100, before_set_brightness_turn_on=False, config_offline_is_off=False): API-Error 429: Rate limit exceeded, retry in 19 seconds.
2023-03-02 09:49:32.168 WARNING (MainThread) [custom_components.govee.light] update failed for 8F:E4:D4:AD:FC:9E:51:22: API-Error 429: Rate limit exceeded, retry in 19 seconds.

I've been playing with this a little this afternoon, and I think it's because I have two instances of the Govee integration - one using my account and one using my son's. When both are active, I get the API Rate Limit errors. If I disable one, the errors go away, but then it seems to get very confused and both devices/entities appear on the one instance (where it should be one on each) and don't function correctly.
It looks like the integration doesn't handle multiple instances well. For now, I'm going to try and move my TV LEDs onto my son's govee account and see if I can manage with just one instance.

For me change poll interval to 60 solved problem. I have 2 govee devices. If someone has more devices, this value i think should be increased to a larger value depending on how many devices You have


GitOps Service: Set GitOps Service replicas to 0 (GITOPSRVCE-801)

JIRA:
https://issues.redhat.com/browse/GITOPSRVCE-801

/retest

/retest

Removed production, and rebased to laster from main branch.

/retest

replaced by https://github.com/redhat-appstudio/infra-deployments/pull/3760


GetUserAsync inconsistent result

I have class for User that extends IdentityUser.
It contains bool property IsEnabled.
In Database i have column IsEnabled with value 1. When i changed this value to 0 while i'm logged in and then i use UserManagers method FindByIdAsync in controller it returns me user from database as is, and IsEnabled property is False. This is correct.
But, when i call the same method with same parameters in the middleware, property IsEnabled is always true. Looks like the manager returns user from cache. Changes are affected when i re-run the solution.
Why this happens in middleware? How to change the behavior?
Invoke method from my middleware contains something like this:
user and dbUser - both have IsEnabled = true. In database is 0.
MyUser user = await _userManager.GetUserAsync(httpContext.User);
MyUser dbUser = null;
if (user != null)
{
dbUser = await _userManager.FindByIdAsync(user.Id.ToString());
}

We need a more detailed repo I'm afraid. Any change you can strip everything down to demo this, and also include the middleware, then tag myself and @HaoK

It looks like you are probably capturing the UserManager in your middleware which results in that capturing the first request's db context which is why you are seeing old data.  For this to work, you will need to service locate (context.RequestServices.GetService<UserManager<TUser>>()) try that to see if that helps

@HaoK was right, Using described solution helps. Thx again.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
21 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
17 seconds
sanity-inspect-image

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
17 seconds
sanity-label-check

✅ Succeeded
16 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
show-summary


allow user to specify additional assembly resolve folders

allow user to specify additional assembly resolve folders on the command line in order to resolve missing assemblies especially when pointing to an ASP.NET project. Resolves issue #4 .

Hey @AndrewKoransky, thank you for your contribution :)
I've merged it as an option to keep in mind for side-loading the source assembly dependencies. I ran into the same issue and solved it in the new release by offloading this task to ILSpy though, obsoleting and removing the AssemblyResolveFolders again. Hope this solves your scenario as well. If not, please let me know.

@h0lg - JFYI... finally got around to needing/testing this new version. Works great! Thanks!


Only call handlers for counters specified in SimpleInstrumentor constructor

When creating a custom performance counter and then registering the handler with PerfItRuntime.HandlerFactories, the new handler will be called for all instances of the SimpleInstrumentor, even if it isn't contained in the original counters property when the SimpleInstrumentor was instantiated.
I have added a simple test to prove the problem, and a fix for the issue.

Thank you for your contribution. This was released as 2.5

Thanks for the quick Merge!


On the mobile terminal, after jumping to the metamask connection, you will be prompted to return to the browser, but you cannot obtain the wallet address and other information after returning to the browser.

On the mobile terminal, after jumping to the metamask connection, you will be prompted to return to the browser, but you cannot obtain the wallet address and other information after returning to the browser.
code:

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.


Created AddressDetailView

Created AddressDetailView.js
Modified User.js, files.js

@blakegreen32 @zahnno please review.


2024.1.2 (seemingly) causes issues with authenitcation.

Version of the custom_component
0.1.9
Describe the bug
Recent Home Assistant core update, 2024.1.2, caused a new notification in HA -
Integration requires reconfiguration, At least one of your integrations requires reconfiguration to continue functioning. Check it out..
Upon clicking, or going to integrations, i can see that 'Southern Company Hacs' has a 'RECONFIGURE' button.
Clicking this shows a window titled 'Southern Company HACS' and two unlabeled boxes.
Assuming that its a login box, i enter my Southern Company login (which i have confirmed to be valid), and then get returned to the same window, with a new red box at the top, with the information Invalid authentication.
Debug log
The log has no reference to this integration within it.

I am getting this issue with 2024.1.1 as well.

I am also getting this error with 2024.1.5

I too am getting this error. I reset my password with Southern Co and still have the same problem. I did not have any issues until the 2024.1.5 update.

I actually got it working by changing my password, deleting the configuration, reinstalling it, then signing in again. I am on 2024.1.5 now and am getting correct reporting.

Unfortunately there is no clean way to remove the integration prior to removing it from HA, as the 'delete' button is missing from the integration for some reason.
The methodology i used was to  -

remove the files from the custom_components directory - note, it doesn't let you remove the directory at this point.
force remove the integration in HACs
restart Home Assistant
verify the component directory is now gone.
go to HACs, search and install again.
prior to restarting HA, verify the files and directory have returned.
restart HA
re-add and re-setup Integration.
quickly go to integrations, find the 'southern company HACS' integration prior to starting, click into it.
delete the devices and entities from the three dots - note, this should be possible without doing all this, but for some reason its not!
go back to intergrations page.
add the integration afresh - this time it now shows the tooltips/descriptions for username/password
notice that it still fails on login with an auth error (even though the username/password are correct).
go to southern website and do a reset.
attempt again

At this point it works again.
No idea why it requires a password reset to start working when the password otherwise works?

My password needed to be reset when I logged in to GA Power to view my info since the integration was "down". So it's hard to tell if it was the integration messing up or Southern Company requiring a mandatory password reset.

Looks like the required password reset is the only issue. I left my integration unchanged, updated my GA power password through the browser, and then reconfigured the integration and was all set.

Checking in here. I think this was resolved in the southern-company-api here:
https://github.com/Southern-Company-HA/southern_company_api/pull/14
I think this can be closed.


feat: block player movement

This PR is part of Cycle's 8 Shape Feature block player movement
Related PRs:
SDK: #991
Protocol: #215

I'm done for the day, tomorrow I'll work on the conflicts with main


Robotium Recorder : Please install a compatible Android API level (15 or higher)

Trying to run a new robotium test I get this error. Does anyone know how to fix it?
I've all the APIs needed. (15 and a lot of higher APIs). What am I missing?
This is part of my gradle file:
`android {
compileSdkVersion 25
buildToolsVersion "25.0.2"
defaultConfig {
    applicationId "com.centervue.exam"
    minSdkVersion 19
    targetSdkVersion 25
    // DO NOT PUT HERE VERSION NAME, THX. Change the Manifest

    testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"

    ndk {
        moduleName "exam_filter"
        ldLibs "log", "jnigraphics"
    }
    renderscriptTargetApi 18
    renderscriptSupportModeEnabled true
}

[...]`
I am testing it on Android API level 28 device. still getting this issue.
Screenshot
StackOverflow Link: https://stackoverflow.com/questions/42908481/robotium-please-install-a-compatible-android-api-level-15-or-higher

Facing the face Issue

I am facing the same issue too.


Update 2-4-4-p4.md

typo in "jQuery"
Purpose of this pull request
This pull request (PR) fixes a type
Affected pages
https://experienceleague.adobe.com/docs/commerce-operations/release/notes/security-patches/2-4-4-p4.html

@tanay-sharma, please be sure to "import" (internal link) this branch into the git.corp repo.

Hi @jeff-matthews, yes, I'll import it. Thanks for the reminder!


Do we need built-in predicates?

Thanks for your hard work in getting this off the ground so far! Looks great!
Out of curiosity, is there any reason you feel like this library needs built-in predicates? It seems to me that the basic building blocks of clojure.spec are specs, regexs (cat, alt, *, etc), data generation and function specs. I know you mentioned JS not having a standard library of predicates like clojure, but I'd argue that that is a good reason to let people bring (or build) their own. The most interesting predicates that make spec valuable beyond a type system are not ones that an easily expressed by a single predicate anyway. Think positive integer, number between, array with a certain length, etc etc.

is there any reason you feel like this library needs built-in predicates

I think you argue correctly that these are not what makes js.spec useful.
However I would like to have a small-ish amount of predicates built in so that js.spec is easier to use and start out with. That would include basic type checking, numbers (positive, nevative with/without zero, number between...) and strings (regex and predefined matchers for email etc.).
In my opinion we get the best of both worlds with this approach: Beginners/curious people can do basic stuff without loading lodash or rolling their own predicates. Everyone else is free to do so if necessary.

Cool, I respect that opinion. I feel like users are going to divided on this issue since a lot of JS devs are hyper-modular and don't want the added weight, but, as you said, the benefits to new users may out-weigh those objections. Thanks for the quick feedback!

a lot of JS devs are hyper-modular and don't want the added weight

Yes, I've been on that side of the fence too. But as a matter of fact you satisfy 1 % of developers and make it worse for everyone else.
I guess we could work complaint-driven here and maintain a separate js.spec-hardcore package without most of the predicates (some are used internally) :grin:

I'd hardly say the pro-tiny-modules camp is 1% of developers, but I get your point. Even if it's something big like 75% of developers, it might still be worth the lower barrier to entry.
One way around this, granted it would require work, would be to use ES6 modules and something like rollup to publish ES6 modules (and CommonJS) and let people tree-shake out the unused code. You could then, conceivably, use lodash/es and then you wouldn't have to create a custom lodash build, either.
At any rate, I'm certainly not pushing for it, I just wanted to open up the conversation. 👍


some ligatures dont work in italics.

Some ligatures from the Operator Mono Lig font don't work in italics in VS Code but seem to be working in comments (which are italic as well.)

Theme: Winter is Coming

Hmm. Are you using the latest version of the script? Also what OS are you using?

Yes, i just installed it a few days ago. I'm on Windows 10.

Ok, I just checked an it's working for me on Windows 10.
BTW: What language setting do you have it set to where <-> is a valid token? Sometime VS Code will treat some character combinations as 2 separate tokens, which may prevent the ligature from displaying. What happens if you select Plain Text?
Here's from Font Preview

And here's VS Code (as Javascript)

It works for me if the font is not in italics, but "Winter is coming" theme puts keyword operators in italics.
For Winter is Coming.

For Cobalt2 (Your theme).

PS - The language is Python (3.9.5).


undefined is not an object (evaluating '_react.PropTypes.any')

This seems to be coming from the gl-react-image dependency, using react native 0.50.0+

do you have a file called Image.js and do you use React.PropTypes ?

This is coming from src/Image.js and lib/Image.js.
Adding prop-types as a peerDependency, importing it into the two files mentioned above, and changing all React.PropTypes to PropTypes fixes the issue.
I can submit a PR if you want, but there's not much to making the changes I've mentioned above.

Those files does not exist in the gl-react-image repo.
@wkoutre if you know of any bugs in other repos, i suggest you create PRs and help out the community!

@stoffern I was going to make a PR on gl-react-image but it’s not the same version being installed when running yarn install gl-react-native as required by react-native-image-croppers.
If you create a dummy project and install gl-react-native, you’ll see what I mean. The src and lib folders are in the the root of the gl-react-image project that also gets installed.
I’ve made a quick script to to replace the correct files locally, but I’ll have more time this weekend to make a PR.

it uses the latest..
https://github.com/stoffern/react-native-image-cropper/blob/master/package.json#L33

@stoffern Please do the following:

react-native init TestProj && cd TestProj
yarn add react-native-image-cropper gl-react gl-react-native
open node_modules/gl-react-image/

In the directories that opens, you'll see lib/ and src/ directories.
src/Image.js:
import GL from "gl-react";
import React, {PropTypes} from "react";
import rectCrop from "rect-crop";
import rectClamp from "rect-clamp";

...

propTypes: {
      source: PropTypes.any.isRequired,
      imageSize: PropTypes.shape({
        width: PropTypes.number.isRequired,
        height: PropTypes.number.isRequired,
      })

...

lib/Image.js (clearly compiled from src):
var _react = require("react");

...

propTypes: {
    source: _react.PropTypes.any.isRequired,
    imageSize: _react.PropTypes.shape({
      width: _react.PropTypes.number.isRequired,
      height: _react.PropTypes.number.isRequired
    }),

gl-react-image/package.json:
{
        "name": "gl-react-image",
        "version": "0.0.2",
        ...
}

This is what I'm referring to, and is 100%  able to be replicated by following the steps above. The issue is in React.PropTypes being deprecated in ~RN 48 or 49. To fix, gl-react-image/package.json needs:
"peerDependencies": {
    ...,
    "prop-types": "^15.6.0"
}

Then src/Image.js needs import PropTypes from "prop-types"; and all its React.PropTypes changed to PropTypes. After that, compile to create an updated lib/Image.js file.

semantic release is broken so thats why there is no new NPM releases and the gl-react-image dep is outdated. will fix it today

should be fixed there when semantic releases new version bb24ac9b3b90d9f414a87f166eb48db4f11bcc9a

Sorry for joining in here late. Posted this and came down with a cold!

new release out:
https://github.com/stoffern/react-native-image-cropper/releases

@wkoutre would we be able to fork gl react image 0.0.2 and make those changes to PropTypes  for react native image cropper?


sts: Expose all flags

To enable customers at large to create STS clusters, we expose all
STS-related flags, along with interactive help.

/lgtm

Thanks @vkareh


Enforce a minimum TLS version

Description
A recent survey indicated that all of the servers that the CM speaks to use TLS v1.2 or newer. Given that we have a code scanning issue indicating that we don't enforce a minimum version, this change refactors the code to require TLS v1.2 at a minimum.
Motivation and Context
Affects: https://github.com/ThePalaceProject/circulation/security/code-scanning/25
Affects: https://www.notion.so/lyrasis/Address-codeql-code-scanning-security-alerts-94b682553e9f43fb98496582278a41b1
How Has This Been Tested?
I ran the test suite, and replaced an existing test with three new tests.
Checklist

[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

I'm usually the last person to allow code to depend on private implementation details, but in this case I'm not sure what choice we have. Without the annotations, the IDE doesn't have a clue what type those parameters/variables have. 😞

My IDE  (PyCharm 2022.3.2) shows an unresolved reference error for ssl._SSLMethod.

I'm about to upgrade from 2022.3.1 to 2022.3.2... I'll check what happens there!

Interesting. Now that I've upgraded to 3.2, I get the same error.
There is a workaround; refer to the type as a fully qualified name in the annotation instead of importing it. That at least turns it into a warning.
Not sure what the right way to handle this is. This is very poor design on the part of ssl.py; they're exposing private types in the public API.

Interesting pycharm isn't resolving that, I think it ships with typeshed stubs, and typeshed defines _SSLMethod https://github.com/python/typeshed/blob/main/stdlib/ssl.pyi#L162-L171

OK, I think this one might be ready to go. We're now testing the SIP client both with and without mocking (both test different aspects of the implementation).


[Updated stubs for API 1827]

Continued in 7145

This PR already merged with PR 3 therefore we are closing it.


Rewrite & cleanup the docs for media events.

Why did integration tests fail?

According to Michał, they just happen to do so every now and then ¯_(ツ)_/¯


fix(expect-revert): fix usage of reason string

Reason string will be encode as output, use bytes(reason string) will not pass the test.

@onbjerg My fault, I found my typo was wrong.


improvement(devtools-browser-extension): Add background color to icon images so they appear well in browser dark mode

New icon:

I'm a bit surprised the color is more purplish than light blue, like the rest of our branding seems to be? But I won't be the one to block it because of that.

I'm a bit surprised the color is more purplish than light blue, like the rest of our branding seems to be? But I won't be the one to block it because of that.

Actually, yeah, I'm curious about that too. @kashms is there a reason for the more purplish hue?


Added SyncTimeout to RedisCacheConnectorOptions

Added SyncTimeout property to RedisCacheConnectorOptions
Modified RedisCacheConnectorOptionsTest.cs to assert SyncTimeout

Resolves Issue: #14

@miclip Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@miclip Thank you for signing the Contributor License Agreement!


Incremental -> counter rename cleanup

Missed a spot with the rename.

Oops, thanks @mmazur

Nice catch!


Bug report - Bulk download fails with revision

eid = 'c51f34d8-42f6-4c9c-bb5b-669fd9c42cd9'
one = ONE()
dsets = one.list_datasets(eid=eid)
one.load_datasets(eid, dsets, download_only=True)

Results in:
one.alf.exceptions.ALFError: No default revision for dataset alf/probe00/pykilosort/clusters.amps.npy
I've temporarily fixed by adding  an assert_unique=False line 1102 of one.api, but I think in this case it should yield the most recent dataset first.

I finally got around to looking into this.  The load methods were never meant to support relative paths as inputs, they just happened to sort of work. I hope you're the only person using it this way.  Because you still stubbornly load things this way I added a bit more support for it although the logic is more complicated now.  If you provide relative paths as inputs there must be no wildcard or regex (it's not possible to properly validate this) and the collection and revision args must be None.  If loading this way you will get a warning when one or more datasets are not default revisions.  Blindly loading an entire session's data is problematic because you get all the revisions and the user has no way of knowing which is the 'correct' version of the data.  For that reason it's better to encourage users to load collections and objects, rather than just passing in the entire output of list_datasets. It's not feasible to serve the most recent data first: if the user explicitly requests data for a list of relative paths they expect it will be returned in the same order.
If you want to download an entire session without old revisions you can do this:
dsets = one.list_datasets(eid, details=True)
dsets = dsets[dsets['default_revision']]['rel_path'].values
assert 'alf/probe00/pykilosort/clusters.amps.npy' not in dsets


MHR Scripts

If I only include dll file from zip, what script should i load for UI to work?

Unclear what you're asking. This also sounds more fit for Discussions.


unweighted bases should be available for array summaries

currently unweighted bases for arrays are skipped in stack.aggregate() if weighted bases are available.
Changing this might also affect qp.ViewManager, qp.ChainManager, ExcelPainter, PowerPointPainter

merged into #1080


Images cannot be empty as documentation suggests

Contributing guidelines

[X] I've read the contributing guidelines and wholeheartedly agree

I've found a bug, and:

[X] The documentation does not mention anything about my problem
[X] There are no open or closed issues that are related to my problem

Description
The documentation describes that images input can be empty so that only tags are generated:
https://github.com/docker/metadata-action/blob/releases/v4/README.md#images-input
However, when actually not defining images, github will complain because the property is set as required and defining it as an empty string will also result in the same error message:
Error: images input required
This can also be observed in the action definiton as required is set to true:
https://github.com/docker/metadata-action/blob/releases/v4/action.yml#L14-L16
Here an example step which will result in the error:
      - id: meta
        uses: docker/metadata-action@v4
        with:
          tags: |
            type=sha

Expected behaviour
The images input is not defined as required and works as the documentation describes
Actual behaviour
Error: images input required
Repository URL
No response
Workflow run URL
No response
YAML workflow
jobs:
  build_and_push_images:
    steps:
      - id: meta
        uses: docker/metadata-action@v4
        with:
          tags: |
            type=sha

Workflow logs
Run docker/metadata-action@v[4](https://github.com/..../..../actions/runs/..../job/....#step:7:4)
  with:
    images:
    tags: type=sha
  type=ref,event=pr
  type=ref,event=branch
  type=semver,pattern={{version}}
  type=semver,pattern={{major}}.{{minor}}
  type=semver,pattern={{major}},enable=true

    context: workflow
    github-token: ***
Error: images input required

BuildKit logs
No response
Additional info
No response

I noticed just now, that the empty images seem to be a feature of v5 of this action which I didn't know exists because I checked for release git tags instead of checking the releases tag.
Sorry


Update office-365-management-activity-api-schema.md

Added details related to SystemSync.

Docs Build status updates of commit a69b0af:
:warning: Validation status: warnings

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:warning:Warning
View
Details

office-365-management-api/office-365-management-activity-api-schema.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 60, Column 2: [Warning: bookmark-not-found] Cannot find bookmark '#SystemSync-schema' in 'office-365-management-activity-api-schema.md', did you mean '#systemsync-schema'?

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8279d56:
:warning: Validation status: warnings

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:warning:Warning
View
Details

office-365-management-api/office-365-management-activity-api-schema.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 60, Column 2: [Warning: bookmark-not-found] Cannot find bookmark '#SystemSync-schema' in 'office-365-management-activity-api-schema.md', did you mean '#systemsync-schema'?
Line 190, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Sounds doesn't work

A bug in the API (always say which version you're using!)

Phaser Labs
no one of these examples doesn't work, throws error in API

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

What's the actual error? because they all run here.

WebAudioSound.js:546 Uncaught TypeError: Failed to execute 'setValueAtTime' on 'AudioParam': The provided float value is non-finite.
    at WebAudioSound.calculateRate (WebAudioSound.js:546)
    at WebAudioSound.set [as rate] (WebAudioSound.js:643)
    at WebAudioSound.applyConfig (BaseSound.js:460)
    at WebAudioSound.applyConfig (WebAudioSound.js:453)
    at WebAudioSound.createAndStartBufferSource (WebAudioSound.js:331)
    at WebAudioSound.play (WebAudioSound.js:207)
    at Scene.create (play audio file.js:31)
    at SceneManager.create (SceneManager.js:544)
    at SceneManager.loadComplete (SceneManager.js:441)
    at LoaderPlugin.emit (index.js:183)

Anyone else get this? I’ve tried 4 different PCs and they all worked.

yes we have discussed about this with @Antriel in slack yesterday

Exact same Chrome build here, plus tested Canary Version 67.0.3390.0 and the labs demos all work using latest 3.4 build that is up there. It doesn't really matter what was discussed on slack, if I don't get more verifications here that this is happening across multiple PCs there's nothing more we can do.

Uncaught TypeError: Failed to execute 'setValueAtTime' on 'AudioParam': The provided float value is non-finite.
    at WebAudioSound.calculateRate (WebAudioSound.js:546)
    at WebAudioSound.set [as rate] (WebAudioSound.js:643)
    at WebAudioSound.applyConfig (BaseSound.js:460)
    at WebAudioSound.applyConfig (WebAudioSound.js:453)
    at WebAudioSound.createAndStartBufferSource (WebAudioSound.js:331)
    at WebAudioSound.play (WebAudioSound.js:207)
    at Scene.create (eval at <anonymous> (frame.html:45), <anonymous>:31:11)
    at SceneManager.create (SceneManager.js:544)
    at SceneManager.loadComplete (SceneManager.js:441)
    at LoaderPlugin.emit (index.js:183)

Chrome Version 65.0.3325.181 (Official Build) (64-bit)
Seems to be coming from https://github.com/photonstorm/phaser/blob/master/src/sound/BaseSound.js#L501 where this.currentConfig.detune is undefined and therefore totalDetune is NaN.

On which example? I tried Audio Sprite and Loop Delay and they both worked. It must be something about the config of a specific example.

Play Audio File

Finally, getting somewhere, thank you.
This is a perfect example of how not to file an issue. Please, don't ever write "none of these examples work" when clearly some do. Link to the specific one you tried that didn't work, as this just wasted all of our time by exaggerating the actual problem.

Sorry for that mistake, what about this error?
BaseSound.js:348 Sound marker name has to be a string!
this is a problem with my own code ?

Not sure re: the marker error. If you are actually using a string for the marker key, then it's a bug. If you're using something else, then use a string.


First draft of Summon command

2 suggestions:

summoner should already be in channel
summoned should not already be in channel

im loving this sudden burst of work coming in 👍 keep it up


Font Weights

Would be useful to be able to select light fonts. I'm looking at something that I'm fairly sure is inaccessible on a webpage, but would meet standards if regular font weight.

@kaylajheffernan Do you have any reference to what is considered to be a "light font"? This spec mentions only the "bold" term.

Hey @moroshko, I'm not sure if there are any accessibility guidelines on light fonts yet, but it's a weigh available in some of the newer fonts (Roboto, Helvetica etc.)
For example this is a normal weight font, and it's accessible

This is a light weight font, I don't know if it's accessible

@kaylajheffernan I see what you are saying, but I'm not sure how can we automate determining whether a certain weight is available or not if there is no clear definition/spec for that.


More (julia) packages to interface with

This is a list to keep track of packages that should be considered in the near future with some notion of priority; feel free to comment with packages that should be considered or shotgunning an interface you'd be happy to help with.
higher priority

[x] Complete interface with DecisionTree.jl e.g. categorical features part
[ ] Impute.jl
[x] Soss.jl at least for basic bayesian models such as bayesian regression or bayesian logreg --> this is happening on branch soss
[x] Finish the interface with Multivariate Stats (e.g. subspace LDA is missing)
[ ]  Complete the interface to Clustering.jl

other models

[ ]  LowRankModels.jl and also this mlj issue
[ ]  ManifoldLearning.jl also this mlj issue
[x] EvoTrees.jl and this mlj issue~~
[ ] tsne
[ ] umap
[ ]  NearestNeighborDescent

[ ] https://github.com/JuliaStats/StatsKit.jl

[ ] https://github.com/Marco-Congedo/PosDefManifoldML.jl note: objects are matrices, could maybe  use the images formalism.

Maybe add to this list unimplemented parts of DecisionTree package, eg the categorical features part.

https://github.com/TensorDecompositions/NTFk.jl (Nonnegative Tensor Factorization + k-means clustering) but GNU licensing
https://github.com/TensorDecompositions/NMFk.jl (Nonnegative Matrix Factorization) also GPL

https://github.com/theogf/AugmentedGaussianProcesses.jl

@OkonSamuel I realize QDA is nowhere to be seen in Julia, maybe that's something you'd be interested in taking up? there's this old package: https://github.com/trthatcher/DiscriminantAnalysis.jl which offers it but I reckon it'd be a fine thing to add to MultivariateStats or have as a standalone model :shrug: ?

@tlienart I'd love to add a QDA model implementation . But https://github.com/trthatcher/DiscriminantAnalysis.jl package seems old (especially julia syntax and standard libraries used) and might not be be compatible with julia 1.0+. A quick look at the compat section in the project.toml file shows this.
Although the dev branch of the repo fixes this and some other compatibility problems.

Yes sorry I should have made this clearer: you  can either

extract  the QDA code and PR it to MultivariateStats
fork the DiscriminantAnalysis package and make it up to date

I think the first option is probably best given that you're already  familiar with the MV package, but if there's a  lot of code to  port  then maybe the ssecond option is better
On  our side if there's QDA it'd be nice (though there soon will be via sklearn)

https://github.com/davidavdav/HClust.jl though would require to basically start anew given it's 4 years  unmaintained

Would be nice to have UMAP, T-SNE and maybe other dimension reduction methods like Phate in MLJ. @ablaom: where would should this be done? In an existing package or a new package. e.g. MLJDimensionReduction.jl?

I see UMAP has a julia interface And there is https://github.com/lejon/TSne.jl, but that's looking pretty old. I'm not sure about Phate. Were you offering to provide direct interfaces to python (ok) or C (better) libraries yourself? These could go in new standalone packages, or you could collect them in a single package if you thought that made sense. Maybe call it DimensionReduction.jl and JuliaAI could host it.


⚠ ClamAV has degraded performance

In 3e3c2a7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 74 ms

Resolved: ClamAV performance has improved in 54f0acd.


add format overview documentation

Adding some documentation to give a high-level overview of JPEG XL: codestream and file format, basic design rationale, features , etc.
This is likely useful as a starting point for anyone new to the codec who wants to start using the libjxl API, understand what the codec can do, or start contributing to the project.

Thanks for all the remarks, @iulia-m-comsa !
Integrated them all.


Cannot set custom userData

In docs there is a way to set custom implementation of userData, but userData is internal and this can not be modified.

Thanks for enlightening a non-obvious point, I'll fix it in the documentation to make it clearer :)
In order to logically separate heaps of parameters into different groups in the config, dsl functions are used, for the current task try wrapping in the context{} function:
TelegramBot("...") {
    context {
        userData = MyImpl()
   }
}

Thanks for the report!

fixed in v5.0.1


Wrong repo url in bower.json

To fix this you'll have to create a ticket here IIRC: https://github.com/bower/registry/issues/

What do you mean by wrong URL? It still resolves, doesn't it?

https://github.com/purescript/purescript-now doesn't exist.

Oh, in the bower.json not the bower registry, I see. That field doesn't mean anything to bower, so it's not a problem really. Although, yeah, it should be fixed for future releases.


Auth: Makes sure refresh token is received from Auth frame

Description
We were creating the refresh token server side, but weren't using it in the client. This is the first step by receiving it from the auth frame.
This also stores the refresh token as a cookie with the correct expiry closing https://linear.app/crossmint/issue/WAL-2905/[refresh-token]-store-refresh-token-in-the-client
Test plan
Tested manually in combo with this pr and JWT is still fetched correctly and now we also receive the refresh token
Package updates
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Amazing PR, I left just some small comments 👍

Thanks! Replied to your comments!


ISPN-8106 Upgrade checkstyle and enable the unused imports check

https://issues.jboss.org/browse/ISPN-8106
Checkstyle 8.1 is much better at recognizing imports needed by javadoc. The only problem I found so far is that Map.Entry#setValue(Object) is not recognized as a reference to java.util.Map.

Looks good! I've been waiting for this all my life!

I'll merge when CI is done with it.

LGTM. Thanks @danberindei !


chore: Add DaFont to frontend/font category

Fixes Issue
Closes #1204

Changes proposed
I added DaFont to the fonts subcategory of the frontend category.

Screenshots

Note to reviewers

@CBID2 @rupali-codes please merge this i'm not getting any option to merge

@CBID2 @rupali-codes please merge this i'm not getting any option to merge

Unfortunately @k-deepak04, I don't have access to Linkshub's Vercel account so we'll have to wait for @rupali-codes' response.

@rupali-codes merge this one too

@CBID2 we have resolved this merge issue before, idk why its coming again 👀


An error during template rendering could throw the server into an infinite loop

We have a call to ctry! within WebPage::into_response, which means that if the rendering of a template fails we call WebPage::into_response on an ErrorPage. I'm not really sure what conditions would have to line up in order for it to happen, but it's possible that the server could get into an infinite loop of rendering and failing templates which wouldn't be good

That could only happen if the error page itself fails to render. Since we test the error page, I don't think this is an issue in practice.

Agreed, it's super unlikely to happen, but it is a possible footgun

Yup, you're right, this sends the server into an infinite loop:
diff --git a/templates/error.html b/templates/error.html
index e7685ec..6a1b855 100644
--- a/templates/error.html
+++ b/templates/error.html
@@ -9,4 +9,5 @@
     <div class="description">
         {{ message | default(value="") }}
     </div>
+    {{ oops }}
 {%- endblock header -%}

and then fetching /not_here.

I think it might honestly better to just panic in that case; that way it only crashes a single request instead of locking up the server. We test errors anyway so it shouldn't ever happen in prod.


libunwind 1.7.2

Noticed that upstream had progressed quite a bit beyond the version that we're packaging here.

The issue on aarch looks like https://github.com/libunwind/libunwind/issues/752

In summary, looks like 1.8.1 is broken (scheduled to be fixed for 1.8.2), but 1.7.2 works. :)


Add .arch function

Note:

I also had a version that wrote out all options

We could also implement a stricter type for Arch

I think we should also implement a catch-all platform (Unknown(String)) to facilitate adding new platforms easily

Just for the record, I've also looked at implementing a custom platform (that would be Custom(String) or so, but didn't yet, because String doesn't implement Copy and so we have to refactor a little more code to use .clone() in some places.
Or is there a copyable string somewhere that we could/should use?


Middleware?

First off, I love the concept and the simplicity of it. I often found myself annoyed with reducers because the project is so small.
With that said, things like redux-sagas and redux-observables seem to offset the need for reducers. Wonder would would be a simple way to introduce a form of middleware without the bloat. My first inclination is composing actions, but not sure if you had a better idea in mind?
I guess I'm looking for a way to do async actions.

Hi @DarrylD
First, thanks for the compliments!
Redux Zero actions are just JavaScript functions. If you want async actions, I don't see why you can't just use async(await) functions.
My concern with adding more stuff to the library, is that one day we end up exactly like Redux, you know?
Thanks for the message!

I'm closing this one. If you have any other questions please feel free to ask.

Valid points. That was my Friday brain talking 😁

@matheusml
Can I use redux middleware? example,  redux-persist

@axetroy could you describe in more details what you mean by that?

@axetroy i think something like that would need to be recreated with actions.
But, I do think it may be a good idea to implement something that allows you to use the many redux middleware libs. Part of the buy in for redux is the ecosystem.

@DarrylD
yeah. redux ecosystem is complete. got many middleware.
If redux-zero can reuse it. that must be awesome and powerful.

@matheusml
I mean, support redux middleware, many should happy to see that.
redux community have a lot of superior middleware.
many of them will not rewrite again just for compatibility with redux-zero.

What you suggesting is something like this:
import createStore from 'redux-zero'

const initialState = { count: 0 }
const store = createStore(initialState, middleware)

export default store

We can have an optional parameter to createStore that accepts external middlewares, but without implementing any specific middleware to Redux Zero.
Is that right?

@matheusml
damn right!:+1:

Seems awesome to me. Any of ( @axetroy, @DarrylD ) you want to take it?
I just created the issue #36

I'm still swamped with work (like the render-prop feature I wanted to tackle 😂). Hopefully, I'll have time soon.

The redux middlewares relies on redux dispatch that's a simple/raw object with type and other inputs.
Middlwares can easily track these dispatch inputs, as well as their types.
Redux-zero don't have dispatchs, it's a simple sotre with setState and getState
I guess it's impossible to port some redux middleware to redux-zero.


Modifying navigation drawer

Hey George,
I had some suggestions regarding nav. drawer-

Increasing height of cover (background image) in navigation drawer

I think (pretty personal opinion), maybe the navigation drawer would look even better if the cover image for Bookmark text was a bit more in height, the way it is in Google apps. I think it's more cozy that way.

Appearance of bookmarks

The bookmarks in nav. drawer appear the same way folders appear in app. Maybe, assigning a different icon for bookmarks and having them appear the same way as different fragments (Settings, About) would make nav. drawer look more consistent.

Material style icon for New(Plus) and Edit(Pencil)

I think, Dir would look even better, if it had the floating circular icon, as in Gmail for compose, for New and Edit(when selected). The same New(Plus) icon may convert into Edit(Pencil) icon on selecting a file or folder.

Adding an option to create a file alongside the option to create folder

As of now, we are able to create new folder with the new folder icon. I think (taking into consideration  above suggestion) adding facility to create new file after tapping the plus icon may add an extra feature.
Many apps have the floating icon, which shows options(which appear as circles above the tapped icon which turns into cross, well you get the point :wink:) when tapped. Maybe the plus icon could be like that.
I'm referencing issue #45 as it is related to this issue.
I hope I didn't do a lot of mutter mutter. :laughing:

Thanks

I'm afraid I'm going to disappoint you on this issue. Specifically:

Increasing height of cover (background image) in navigation drawer
Appearance of bookmarks

For these two, I've replied in #45.

Material style icon for New(Plus) and Edit(Pencil)

I'm assuming you're referring to the Floating Action Button. According to the Material design guidelines, "A floating action button represents the primary action in an application". Dir being a file manager, there is no inherent primary action (or at least it's not "Create directory"). The Pencil/operations action could be a candidate but I'm afraid it'll make for a unnecessarily separated set of actions. I've actually been thinking about completely disbanding the operations "group" and having them on the top level when long pressing, as quite a few users have had trouble finding these; making it a FAB will make that transition even harder.

Adding an option to create a file alongside the option to create folder

I was actually thinking about doing this during the Material redesign, but even if there's a "create file" action, it's not apparent what type that file should be. Given the options, I opted to not add it at all to avoid having a half-baked implementation and UX. (FWIW, given the time constraints, the options where either creating only text files, which would probably also need a text editor implementation, or allowing any filetype but being unable to edit almost all of them in-app).
As a side note, (and this is the case both for the FAB and the nav drawer), many apps try to fit these patterns everywhere, sometimes worsening the UX, just because they feel it's a must-have for Material design apps. In designing Dir I've deliberately avoided using standard components that make no sense for the app's purpose and use case only to make Dir feel more "standard". Hence the lack of some ubiquitous components/patterns :)
Hope this makes sense,
George.

Hey George,
It's very nice to have well reasoned reply from you. Thanks for that.
I don't mean to be persistent here, but think I need to clarify some issues listed above, that I think, I didn't explain well.

Appearance of bookmarks

I think one may want to name a bookmark something different than the original name of the folder. So, maybe it might be that bookmarks and their original folder could be discriminated. This is also the reason I mentioned making bookmarks look similar to other fragments in nav drawer.

Adding an option to create a file alongside the option to create folder

By allowing users to create new file, I meant to allow them to create any kind of new file. When a user shall type name for that file, the user would type the name with extension, which is how the file type shall be determined. Many apps do similar, I guess.
All other issues mentioned above are perfectly handled by you. Thanks for such a well thought reply.

No worries, thanks for the update.
About the bookmarks, what do you mean by making them look similar to other fragments? They're actually using the same code behind the scenes at the moment. In regards to renaming them, this actually could be done fairly easily using the current interface and code.
In regards to the file creation option, I haven't used other file managers for a long time ago. Would you happen to know how the most popular ones handle this? If it turns out that enough popular ones do it that it's expected, I would be up for adding this generic "create file" option, and let opening the file worry the apps we delegate to.


[Page UP and donate button] [Fuctional] [Desktop] The Up button is shown up on the second screen

Environment: Windows 11, Chrome, 123.0.6312.59 (64 bit), screen resolutions: 1280x1024, 1024x768, 800x600.
Reproducible: always.
Build found: https://github.com/ita-social-projects/StreetCode/commit/a7d807fdd928985083647a72ec61fa983110fea6
Preconditions
Open any Streetcode.
Steps to reproduce

Choose resolution 1280x1024.
Scroll down to the second screen.
Pay attention to the UP button appearance.

Actual result
The Up button is shown up on the second screen.
https://github.com/ita-social-projects/StreetCode/assets/86358268/ee913ee5-236f-4a57-98ba-46a8d24f6ea8
Expected result
The Up button is shown up after the second screen (Text and video block).
User story and test case links
User story #78 #24
Test case #93

It is not considered a bug


TODO: Track session data modifications.

Track session data modifications like Flask do.

We can use SessionMixin from flask.
Check out: flask/src/flask/sessions.py at 3.0.x · pallets/flask

We can use SessionMixin from flask. Check out: flask/src/flask/sessions.py at 3.0.x · pallets/flask
There are also some good ideas at encode/starlette#2019


More doc updates

Added a full-config.md page.  Could really use some help listing all the default values for items that are not undefined or false by default.

This concludes my full parsing of the entirety of the docs.  Added a few new pages, mostly just styled/restructured things (bullet points, numbered items, page breaks).  Changed present tense to future tense.


pdb_tidy behaviour

version 2

pdb_tidy does not remove previous END lines, resulting in doubled END statements.

ATOM     16  O   ASN A   1      21.581  38.297  -2.176  1.00  0.00           O
TER      17      ASN A   1
END
END

Regardless the above mentioned bug, I agree with the current behavior of pdb_tidy (version2), namely the way it handles sequential chains and split chains with gaps and unsorted residues. @JoaoRodrigues could you run pdb_tidy on tests/data/full_example.pdb on version2-unittest branch to see if you agree with its behavior also?

I had an issue with how I found 'END' statements .. trying to match it after doing 'line.strip()' and waiting for a space anyway. Nice catch. Fixed.


RFE: Copy user on CMD + /

It would be awesome if you can add the possibility to copy the user / username field from a password file. This makes a lot of sense if you don't want to use a browser plugin, which might be a security concern in some cases.
What do you say ?

I'd be open to this feature if you're looking to contribute, happy to talk over proposals

Can I have your mail address so we can sync easier?

Sent you a message on linkedin with it!


Cotary Create: Minimum DAO Bounty cannot be set to 0

"If Auto-incentivize proposal curation is off, set Minimum DAO Bounty & DAO Bounty Const to 0"

DAO Bounty Const isn't being set with this setting and according to the contract both Minimum DAO Bounty and DAO Bounty Const cannot be set to 0.

I've created issues for this in the repo, but haven't heard a response:
https://github.com/daostack/infra/issues/76
https://github.com/daostack/infra/issues/75
I think we should make the change ourselves and PR it in... until then we should enforce this in the form library :/ we can set it to 1 instead of 0


Add code for metadata usage in downstream tools

create code files that can be used in downstream tools
add make target to generate

Managing the descriptions and titles here -
https://aquasecurity-my.sharepoint.com/:x:/g/personal/owen_rumney_aquasec_com/EX9vHs9i0XlJouKxPO3zO24BQooz85dZyEOmaYOFOkx1DQ?e=dsV1XQ


Recommendation to remove a default secret key

Hello,
We are a cybersecurity research group from the CISPA Helmholtz Center for Information Security and Ca’ Foscari University of Venice. We recently conducted an analysis of the session management used in your web application as part of our research. We have discovered a security flaw that we believe requires your attention.
Description:
After our analysis, we have identified that your application is using a hard-coded secret key that is leaked through GitHub. If operators who install your web application do not change this secret key, they are vulnerable to cookie forgeries. The cookie forgery attack allows an attacker, knowing the key used to sign a cookie, to forge new arbitrary cookies to impersonate and take over other accounts.
link to the relevant code part
Recommendation for Mitigation:
To address this issue and enhance the security posture of your web application, we highly recommend setting the secret key from an environment variable. If the environment file is not shared on GitHub then this would force operators of your application to create their own key upon installation, forcing them to set their own secure secret key.
We hope this notification helps improve your security and would appreciate if you could take a look into the relevant code parts and comment on the issue.
Thank you!

I like this idea (because I suggested the same 3 years ago in #931).

In PDANext we have addressed this issue, it is now set via an environment variable, also I have a python script that generates  the .env file which generates a new SECRET_KEY on each run of the script. This can then be run via docker on load.
SECURITY WARNING: keep the secret key used in production secret!
SECRET_KEY = env("SECRET_KEY", default="")
This is the new code in PDANext, which is also based on Django instead of flask.
We also as you mentioned is that it is in the install instructions that you need to generate a new key.


homebrew install jemalloc failed

==> cmake ../third-party -DUSE_BUNDLED_BUSTED=OFF -DUSE_EXISTING_SRC_DIR=ON -DCMAKE_C_FLAGS_RELEASE= -DCMAKE_CXX_FLAGS_RELEASE= -DCMAKE_INSTALL_PREFIX=/usr/lo
==> make VERBOSE=1
Last 15 lines from /Users/nano/Library/Logs/Homebrew/neovim/02.make:
/usr/local/Cellar/cmake/3.3.2/bin/cmake -E touch /tmp/neovim20151109-1839-jztr02/neovim-0.1.0/deps-build/build/src/jemalloc-stamp/jemalloc-update
[  7%] Performing configure step for 'jemalloc'
cd /tmp/neovim20151109-1839-jztr02/neovim-0.1.0/deps-build/build/src/jemalloc && /tmp/neovim20151109-1839-jztr02/neovim-0.1.0/deps-build/build/src/jemalloc/configure CC=/usr/local/Library/ENV/4.3/clang --prefix=/tmp/neovim20151109-1839-jztr02/neovim-0.1.0/deps-build/usr
checking for xsltproc... /usr/bin/xsltproc
checking for gcc... /usr/local/Library/ENV/4.3/clang
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... configure: error: in /tmp/neovim20151109-1839-jztr02/neovim-0.1.0/deps-build/build/src/jemalloc': configure: error: cannot run C compiled programs. If you meant to cross compile, use --host'.
See `config.log' for more details
make[2]: *** [build/src/jemalloc-stamp/jemalloc-configure] Error 1
make[1]: *** [CMakeFiles/jemalloc.dir/all] Error 2
make: *** [all] Error 2

cannot run C compiled programs is a severe issue with your environment or homebrew setup.
See https://github.com/neovim/homebrew-neovim/issues/110 , https://github.com/neovim/homebrew-neovim/issues/108 , https://github.com/neovim/homebrew-neovim/issues/95 , https://github.com/neovim/homebrew-neovim/issues/89


refactor: Remove MySQL related options from Datanode

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

remove mysql_addr and mysql_runtime_size in datanode/src/datanode.rs
remove command line argument mysql_addr in cmd/src/datanode.rs

Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/issues/1739

It seems it will break existing tools. gtctl may have the same issue. Cc @zyy17

lgtm


Tajp/wasm via scalar spike

Oooh it's much faster if we only migrate the x4 values...

Sorry - I accidentally created this PR to upstream instead of my fork.


ES-related changes for upgrate to 7.3.2

Made changes based on SearchGuard's changes for ES version upgrade to 7.3.2

Please update UT results for security and security advanced module.


Add host option

Allow users to specify the host. This is helpful when using docker-compose and you're running inside of a container and dynamodb-local is a linked service and you need to specify the service alias as the host.

Addresses https://github.com/99xt/serverless-dynamodb-local/issues/87

@rehrumesh can we get this merged? This would be super helpful!

Can we merge this, please?

This is great. Thanks for the PR @smdern . Can you also update the documentation accordingly?
@deanmraz @kinyat NPM is updated. Apologies for the delay.

Know this is closed, but the usage is if I see correctly on my serverless.yml
custom:
  dynamodb:
    stages:
      - dev
      - local
    start:
      port : 8000
      shareDb: true
      migrate: true
      noStart: true
      host: dockerized-dynamo

or I have to forget about the noStart?


Move ReadingBlockHitShape back into Voicing

I had thought that the readingBlockHitShape property of ReadingBlock was specific to ReadingBlocks, but it is needed for other things that mix Voicing. It should be moved back into Voicing, and work for both ReadingBlock highlights and Interactive Highlights.
This is needed of Voicing work going on now in RaP, with AccordionBox.

This was done as part of #1217, closing.


[BUG] Audio doesn't work after ongoing was meeting was interrupted through a phone call

Describe the bug
If user receives phone call during meeting, user keeps teams call on hold and receives phone call, after the phone call ends and user resumes the call, input and output audio doesn't work.
To Reproduce

User joins a scheduled meeting using url and token as a guest using his iPhone through my app XYZ.
Host joins using Teams app on his/her desktop.
Host admits guest.
User receives a phone call.
User keeps meeting on hold on app.
User accepts phone call and answers.
User ends phone call.
User returns to app XYZ and resumes Teams meeting call.
User cannot hear anything, host cannot hear user.

Setup (please complete the following information):

OS: [e.g. iOS] iOS 15.2
IDE : [e.g. IntelliJ]
Version of the Library used:    AzureCommunicationCalling - 2.0.0
AzureCommunicationCommon - 1.0.2

Additional context
didChangeSpeakingState() call back is received after the when host speaks, but user cannot hear anything.
Any suggestions appreceited.
Thanks,

@mariusu-msft for triage.


Markdown preview now detects local files better

Implemented image preview for local image file in markdown.
Implemented opening local files by markdown links.

@JasonStein this implements "broadFileSystemAccess" access too. So no need to merge it right away.


bug: log streams properly end when internal stream closes

Changes <EventStream as Stream>::poll_next to properly end the stream when the internal stream ends
Need to consider, how do we pass permanent/temporary failures from Provider -> FilterWatcher -> EventStream

Merged - opened a new issue to track the actual problem


Infer block_count from superblock if not provided in config.

This pull request attempts to autodetect the block_count from the superblock if not provided in lfs_config (i.e. cfg->block_count == 0).
This PR slightly increases RAM usage by increasing the lfs_t struct by 4 bytes.
I think this PR could also negatively impact mounting performance (I refactored it so that the superblock search and the gstate search are independent), but I think those are super fast anyways. A possible organization improvement would be to abstract the traversal to another function, but that might reduce readability. We can leave that for another PR.
~@geky currently this is set as a draft as I want to write more unit tests, but need some advice on how to debug. Currently one of the test permutations hangs; I don't quite understand what the permutations are or how to debug.~
Related issues & PRs: #865, #753
For ease of review, the biggest commit (73285278b98e0b15a608325b3f7e4c5b8e0c6b75) just moves the identical traversal into 2 functions (superblock, gstate), and moves the superblock validation to a separate function. There's no real logic change in that PR, except that the superblock is found first, then the gstate is updated (rather than at the same time).

@geky I think this is ready now!

what are our thoughts on this? It's less invasive than the other PRs. It's also a backwards compatible change. Any future changes (such as if we allow lfs_config_t to be mutable) are compatible with this PR. In that example, we'd just remove the newly introduced lfs->block_count field and have all the pointers point back to the now mutable lfs->config->block_count.
I'm looking forward to some form of block_count autodetection so that I can publish my cli tool that manages littlefs partitions to PyPI.

Hi @BrianPugh, thanks for this. I left some feedback.
I'm not sure fetching the block_count from disk during lfs_format makes sense. I left an in-code comment about that.
I think we should also add block_count (and maybe block_size for completeness) to lfs_fs_stat, so the block_count is available to the user after mounting. This should just mean another assignment in lfs_fs_rawstat: here.
I'm also curious if the test_superblocks_fewer_blocks and test_superblocks_more_blocks tests from #753 can be ported over to this. They provide coverage for a few more corner cases, and should cover changes to lfs_fs_stat well enough.

I'm also thinking I'll try to get lfs_fs_grow (#753, #702) in on the same minor release as this, since they are closely related. Though that doesn't affect this PR.

@geky currently this is set as a draft as I want to write more unit tests, but need some advice on how to debug. Currently one of the test permutations hangs; I don't quite understand what the permutations are or how to debug.

Were you able to figure out debugging with the test framework? It's unfortunately a bit dense not documented well.
If a test permutation fails in CI:
tests/test_compat.toml:1296:failure: test_compat_major_incompat:1g12gg2 (PROG_SIZE=16, BLOCK_SIZE=512) failed

You should be able to rerun it locally like so:
# build the test runner
$ make test-runner -j
# run the failing test
$ ./scripts/test.py runners/test_runner test_compat_major_incompat:1g12gg2

Though for the Valgrind tests you need to pass the valgrind flag:
$ ./scripts/test.py runners/test_runner test_compat_major_incompat:1g12gg2 --valgrind

Note the test framework expects quite a bit more from the dev environment than littlefs itself. It's unlikely to work with a non-GCC/Clang compiler...

I'll add the lfs_fs_stat  functionality and investigate the unit tests you recommended

@geky-bot copying over the tests from #753 ALMOST works, but it would also involve copying over all the runner and related changes. I feel like that might balloon this PR and make a bunch of unnecessary merge conflicts. Waiting for your advice before performing more actions.
Since your last review I:

Updated lfs_fs_stat along with the related struct fields.
Fixed linting.

so whats the conclusion on this? Does the PR seem good? Waiting to bundle it with other PRs for a release?

This is looking good, thanks for the updates. I'm currently planning to bring this in on the next minor release.
I'm still wanting to look into the extra tests in https://github.com/littlefs-project/littlefs/pull/753, which as you point likely come with baggage, and lfs_fs_grow (https://github.com/littlefs-project/littlefs/pull/753, https://github.com/littlefs-project/littlefs/pull/702) since it is closely related. But I think these would be more appropriate in separate PRs.

I've gone ahead and reverted the lfs_scan_for_superblocks and lfs_scan_for_state_updates refactor. The new code path would have led to scanning the superblock chain twice, when we only need to scan it once.
Let me know if you see anything wrong with this.

@geky that's fine by me since we no longer invoke lfs_scan_for_superblocks at multiple places 👍

I've also gone ahead and adopted the block-device/runner changes in https://github.com/littlefs-project/littlefs/pull/753. I realize this is probably pushing the scope of this PR, but it lets us adopt the tests in https://github.com/littlefs-project/littlefs/pull/753, and is probably a good long-term change.
I also tweaked lfs_fsinfo a bit, mainly to make the struct fields follow the order found in the superblock for consistency. I realize this is really nitpicky.
If you're ok with these changes this looks good to me for the next minor release.

Also the now-dependent lfs_fs_grow API is here: https://github.com/littlefs-project/littlefs/pull/872

all seems good to me!

This is on master now, thanks for the PR!

Whoops, meant to respond to this one: https://github.com/littlefs-project/littlefs/pull/886. Well this is technically on master as well so...


AxisLimitsChanged event: the emitted RenderDetails contains old information

Question: Why are the RenderDetails one render behind when implementing an AxisChanged event handler? Is this a WPF only issue?
ScottPlot Version: current github main branch
Code Sample:
In the following code sample (modified from the WPF sandbox from the repo) I have subscribed to the AxisLimitsCHanged event with my AxisChanged handler. In my handler I set the title of the plot to show the "AxisLimits" from the RenderDetails object.
using System.Linq;
using System.Windows;
using ScottPlot;

namespace Sandbox.WPF;

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
        WpfPlot1.Plot.RenderManager.AxisLimitsChanged += AxisChanged;
        double[] Xs = Enumerable.Range(1, 5000).Select(i => (double)i / 1000).ToArray();
        var signal = WpfPlot1.Plot.Add.SignalXY(Xs, Generate.Sin(count: 5000, oscillations: 4));

        void AxisChanged(object sender, RenderDetails e)
        {
            var changedPlot = (ScottPlot.Plot)sender;
            var newLimits = e.AxisLimits;
            WpfPlot1.Plot.Title($"New Y limits: {newLimits.Bottom:F1}, {newLimits.Top:F1} New X limits: {newLimits.Left:F1}, {newLimits.Right:F1}");
        }
    }
}

When first rendered, the AxisChanged event must have been called during initialisation and plotting of the data, and the title has the correct limits:

But if I zoom out once with the mousewheel the axis limits in the title are old (note: I had to screengrab this, because a copy to clip board from the right click context menu would force another render and the saved image title is then correct!):

If I then reset zoom with mouse wheel click
the graph resets correctly, but the axis title shows the previous zoom extent:

I require a second click of the mouse wheel to force another zoom to extents to see the correct limits in the title:

Context to Question
I intend to update RenderDetails.cs to store the previous render's axis limits
    /// <summary>
    /// Axis limits of the primary axes for the previous render
    /// </summary>
    public readonly AxisLimits PreviousAxisLimits;

Which would be populated inthe constructor:
        PreviousAxisLimits = lastRender.AxisLimits;

This would then allow the nature of the AxisLimitsChange to be established within the AxisLimitsChanged event handler.

It looks like RenderDetails contains current information afterall, but my attempt to see the latest information by setting the plot title was illinformed.
The AxisChanged event is happening after the render has occurred, and there is no new render when the title (or xlabel/ylabel) are set.
Should setting the labels automatically trigger a new render, or do we expect user to call their own render?
I've tried adding a call of WpfPlot1.Plot.Render() after setting the title, but even though I can see the renderCount is incremented, the title is still not updated.
Which render action would be responsible for setting the title?
    public static List<IRenderAction> DefaultRenderActions => new()
    {
        new RenderActions.PreRenderLock(),
        new RenderActions.ClearCanvas(),
        new RenderActions.RenderFigureBackground(),
        new RenderActions.ReplaceNullAxesWithDefaults(),
        new RenderActions.AutoScaleUnsetAxes(),
        new RenderActions.ExecutePlottableAxisManagers(),
        new RenderActions.ApplyAxisRulesBeforeLayout(),
        new RenderActions.CalculateLayout(),
        new RenderActions.ApplyAxisRulesAfterLayout(),
        new RenderActions.RegenerateTicks(),
        new RenderActions.RenderStartingEvent(),
        new RenderActions.RenderDataBackground(),
        new RenderActions.RenderGridsBelowPlottables(),
        new RenderActions.RenderPlottables(),
        new RenderActions.RenderGridsAbovePlottables(),
        new RenderActions.RenderLegends(),
        new RenderActions.RenderPanels(),
        new RenderActions.RenderZoomRectangle(),
        new RenderActions.SyncGLPlottables(),
        new RenderActions.RenderPlottablesLast(),
        new RenderActions.RenderBenchmark(),
    };

resolved in #3525


Nitpicking: fix JavaScript case

Stumbled upon "Javascript" on https://learn.microsoft.com/en-us/windows/apps/winui/ and went for a larger fix to change such occurrences across the repo.

Learn Build status updates of commit ee1ab29:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

hub/apps/performance/choose-between-tools.md
:white_check_mark:Succeeded

hub/apps/winui/index.md
:white_check_mark:Succeeded

uwp/get-started/create-a-basic-windows-10-app-in-cpp.md
:white_check_mark:Succeeded

uwp/get-started/universal-application-platform-guide.md
:white_check_mark:Succeeded

uwp/launch-resume/how-to-create-an-extension.md
:white_check_mark:Succeeded

uwp/monetize/adcontrol-in-html-5-and-javascript.md
:white_check_mark:Succeeded

uwp/monetize/banner-ads.md
:white_check_mark:Succeeded

uwp/monetize/error-handling-in-javascript-walkthrough.md
:white_check_mark:Succeeded

uwp/monetize/install-the-microsoft-advertising-libraries.md
:white_check_mark:Succeeded

uwp/monetize/set-up-ad-units-in-your-app.md
:white_check_mark:Succeeded

uwp/networking/web-feeds.md
:white_check_mark:Succeeded

uwp/whats-new/windows-10-build-10586.md
:white_check_mark:Succeeded

uwp/winrt-components/creating-windows-runtime-components-in-cpp.md
:white_check_mark:Succeeded

uwp/xbox-apps/getting-started.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Cannot give access to "Services: Unbound DNS: Overrides"

Hello,
I created a user in the local database, and gave him the "Services: Unbound DNS: Edit Host" privilege.
Then i have 2 issues:

First the Unbound item does not show up in the menu. And even if i check all Unbound related privileges for this user, "Unbound: Overrides" still does not show up.
Second, if i go directly on https://router.ubicast.net:8443/services_unbound_overrides.php it redirects me on https://router.ubicast.net:8443/services_unbound_host_edit.php so thu user cannot see/edit the existing overrides.

Thank you for your help.

Hi @nikaro,
Thanks for the report. Looks like there are multiple oddities in the ACL setup. This patch will help:
# opnsense-patch  c313826

Cheers,
Franco
[1] https://github.com/opnsense/core/commit/c3138260e0465603a10af4b8d6cf00dde6bab039

I confirm that it works. Thank you very much.
PS: very nice this opnsense-patch thing.

Thanks for confirming. I'll have this shipped in 18.7.1 next week :)
Cheers,
Franco


e2e GCSMove Test Scenarios

@itsankit-google @rmstar  Please review
cc @sawantpritam

Please resolve conflicts

squashed commits and resolved conflicts

Please take a look at the test failures

Please take a look at the test failures
Tests are failing with sandbox OOM issue     https://cdap.atlassian.net/browse/CDAP-19026

@rmstar  "Sandbox OOM issue is resolved. Commits Squashed and conflicts resolved"

Please resolve conflicts


After building a new image nodes keep crashing

Hi @colinmollenhour,
I already have used your repo for production and for a client before. Today I built an image with the help of your makefile I wanted to update the content for more recent version.
I had to solve some errors but the image builds OK. Here is the Dockerfile I modified (added mariadb-backup).
 Those keep crashing and being respawned.
time="2021-07-12T10:26:55.771369100+02:00" level=info msg="NetworkDB stats DESKTOP-9RB1276(cab8f4fe809e) - netID:ase91jt88x6l7xrcwltm2lnte leaving:false netPeers:1 entries:2 Queue qLen:0 netMsg/s:0"
time="2021-07-12T10:26:55.771645500+02:00" level=info msg="NetworkDB stats DESKTOP-9RB1276(cab8f4fe809e) - netID:jyro214m0o82tjg0zmp7ly46v leaving:false netPeers:1 entries:23 Queue qLen:0 netMsg/s:0"
time="2021-07-12T10:27:43.830498300+02:00" level=info msg="ignoring event" container=890fae1c73f80299b8c8de1a7d5a435ba385b98e6d01d942292013cefafa1818 module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
time="2021-07-12T10:27:43.832897700+02:00" level=info msg="shim disconnected" id=890fae1c73f80299b8c8de1a7d5a435ba385b98e6d01d942292013cefafa1818
time="2021-07-12T10:27:43.833261100+02:00" level=error msg="copy shim log" error="read /proc/self/fd/16: file already closed"
time="2021-07-12T10:27:43.838691600+02:00" level=warning msg="rmServiceBinding 48749d62372c8340253d22209c89863beb1eeaefcdbd4b42fd34b9c4d04a8f97 possible transient state ok:false entries:0 set:false "
time="2021-07-12T10:27:50.002461500+02:00" level=info msg="starting signal loop" namespace=moby path=/run/docker/containerd/daemon/io.containerd.runtime.v2.task/moby/f0c91cfde6c49952d75c53b2bf1cac60b35d8e05ec3c21d8890e70dea80228e7 pid=19134
time="2021-07-12T10:28:03.470156800+02:00" level=info msg="ignoring event" container=aef28517c12dcd4d9a753f27b1a3039195c703816306c54b03e3365957d43efa module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
time="2021-07-12T10:28:03.471073400+02:00" level=info msg="shim disconnected" id=aef28517c12dcd4d9a753f27b1a3039195c703816306c54b03e3365957d43efa
time="2021-07-12T10:28:03.471412400+02:00" level=error msg="copy shim log" error="read /proc/self/fd/12: file already closed"
time="2021-07-12T10:28:03.486098500+02:00" level=warning msg="rmServiceBinding c976f03a1e46a22396eeb60845adc9a161c8046eee85155562d7bd9c013603a1 possible transient state ok:false entries:0 set:false "
time="2021-07-12T10:28:10.035381300+02:00" level=info msg="starting signal loop" namespace=moby path=/run/docker/containerd/daemon/io.containerd.runtime.v2.task/moby/69adfa9a6ceaf532a8f747bf0f8b7c0aadd6869b06e717c528878828029c7333 pid=19488
time="2021-07-12T10:28:14.928939700+02:00" level=info msg="ignoring event" container=24cc8da34090598a7afbca1838d2d679de6028e640f75d491f870f84d5eb11ce module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
time="2021-07-12T10:28:14.941253600+02:00" level=info msg="shim disconnected" id=24cc8da34090598a7afbca1838d2d679de6028e640f75d491f870f84d5eb11ce
time="2021-07-12T10:28:14.941468900+02:00" level=error msg="copy shim log" error="read /proc/self/fd/19: file already closed"
time="2021-07-12T10:28:14.947387600+02:00" level=warning msg="rmServiceBinding 2726957a70a89cd28dd7c85cefa4585fccc9f067c85a778a99ec4a441328a5b9 possible transient state ok:false entries:0 set:false "
time="2021-07-12T10:28:21.119924800+02:00" level=info msg="starting signal loop" namespace=moby path=/run/docker/containerd/daemon/io.containerd.runtime.v2.task/moby/e7ec826bbbbc31ed1628332a2c2356213feca930f04f9f69c94b954779e53344 pid=19751

I collected the log file, but I don't know the reason it's crashing. If you have any idea.
The workaround is to set the version with bionic (10.2-bionic, 10.3-bionic...), but this won't last long I'm afraid.

Hey sorry for the slow reply, I didn't see this one.. I couldn't reproduce the issue.

That's ok thanks for the reply anyway.
Emeric
Envoyé depuis Yahoo Mail pour Android
Le ven., juil. 1, 2022 à 1:56, Colin @.***> a écrit:
Closed #106 as completed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Nzbgeek empty nzbs to SAB

Seems anything I snatch from the nzb search page to SAB from api.nzbgeek.info sends an empty nzb to it.

Nothing i can do regarding this. All its does does is passing the download url to sabznbd. You need to manually download that file your self and check if there is any content in the nzb file. If it is its a sab issue, if not its a nzbgeek issue.

Ok I tested the link in a browser and the nzb downloads and saves to watch folder, sab works.
Added the link manually in sab- get nzb and it gets added too.
Is this a bug in the sab api? I wonder what's different to the nzbgeek url compared to other newznabs which work ok?

Hmm. I'll see if I am a member there any check. Can you give me the link to nzb you download? You can send it as a pm on the kodi forum if you prefer.

Thanks, if you need my login let me know. It happens with any nzb so not really relevant? This is the log from Sab if its any help:
2015-08-21 18:38:26,224::DEBUG::[interface:421] API-call from ::1 [Python-urllib/2.7] {'output': 'json', 'cat': 'tv', 'apikey': '123456789', 'mode': 'addurl', 'name': 'https://api.nzbgeek.info/api?t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789'}
2015-08-21 18:38:26,226::INFO::[init:513] Fetching https://api.nzbgeek.info/api?t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789
2015-08-21 18:38:26,227::INFO::[nzbqueue:218] Saving queue
2015-08-21 18:38:26,229::DEBUG::[init:844] Saving data for SABnzbd_nzo_rz_uzu in C:\Program Files (x86)\SABnzbd\admin\future
2015-08-21 18:38:26,230::INFO::[init:919] Saving data for queue9.sab in C:\Program Files (x86)\SABnzbd\admin\queue9.sab
2015-08-21 18:38:26,232::INFO::[urlgrabber:116] Grabbing URL https://api.nzbgeek.info/api?t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789
2015-08-21 18:38:26,448::INFO::[misc:811] Creating directories: M:\Incoming\NZB\Temp\api!t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789
2015-08-21 18:38:26,453::WARNING::[nzbstuff:744] Empty NZB file api?t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789.nzb [https://api.nzbgeek.info/api?t=get&id=582208513c3c21003471382d208a86fd&apikey=123456789]
2015-08-21 18:38:26,457::INFO::[init:908] C:\Program Files (x86)\SABnzbd\admin\future\SABnzbd_nzo_rz_uzu removed
2015-08-21 18:38:26,457::INFO::[nzbqueue:218] Saving queue
2015-08-21 18:38:26,459::INFO::[init:919] Saving data for queue9.sab in C:\Program Files (x86)\SABnzbd\admin\queue9.sab
2015-08-21 18:38:29,438::DEBUG::[bpsmeter:253] bps: 3.1966481481
2015-08-21 18:38:34,440::DEBUG::[bpsmeter:253] bps: 1.28444708631

Not as it happens with everyone.

Any luck with this?

Sorry, no i havnt had the time.


When scrolled to the top of a page and opening a modal the page content shifts down about 10px

To reproduce:
Go to documentation: Modals Docs
Make sure you are scrolled all the way up and press the button to open a modal.
All the content except the navbar will shift downwards.

Fixed in v0.4.4-alpha

Confirmed works.


Add proxy (with/without authentication) support

 Issue by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT
Originally opened as https://github.com/twogee/missing-link/issues/11

Add proxy (with/without authentication) support

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 20 Mar 2011 at 5:37

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT

Original comment by<EMAIL_ADDRESS>on 20 Mar 2011 at 5:38

Added labels: Project-ant-http

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT

Original comment by<EMAIL_ADDRESS>on 29 Mar 2011 at 11:17

Added labels: Milestone-1.2
Removed labels: Milestone-1.1


any intention of supporting web assembly in nodeos? tccboot maybe not applicable

https://bellard.org/tcc/tccboot.html this is dated but I do like the concept :-) ;-)

Web Assembly support is dependent of support done by Node.js itself, that's already working on it.

Web assembly requires emscripten / clang / llvm support, does nodes intend
to be self hosting?
On Saturday, March 3, 2018, Jesús Leganés-Combarro<EMAIL_ADDRESS>wrote:

Closed #398 https://github.com/NodeOS/NodeOS/issues/398.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/398#event-1502478901, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK2HhIGmKDM36vVKEqoWqt9logEJ4Eprks5tanpLgaJpZM4SasLT
.

--
Thanks In Advance
Kind Regards
Laust Frederiksen

does nodes intend to be self hosting?

Yes, it does, but it's not possible at this moment. Would you be interested to help on this area? What do you suggest?


Update Traction Tenant UI for Basic Messaging (chat bubble style)

Update the Traction Tenant UI as per this mockup:
https://xd.adobe.com/view/6de87d50-b22b-4f4f-a32c-8cadb2997a25-38bb/
AC:

On the contact rows there's a button that will nav you to a messages page that it contact-centric for that selected contact (ie https://xd.adobe.com/view/6de87d50-b22b-4f4f-a32c-8cadb2997a25-38bb/screen/2b763930-666d-45cc-90d1-f03b0b1b621d/)
The message page for the contact shows the view from the UX mockups
You can send messages
Receiving would either be a refresh button (or polling), or just have to go back to the page again. Can probably leave that for later. There are no webhook listens for tenants to auto-pop a message reception.
When you go to "Messages" in the nav bar it has some view of all your messages, either sorted by contact, or just all messages. I don't see any mockups for that so you will need to consult with Guru on what that is supposed to do.

Note the "is typing" part at the bottom is impossible. So leave that out

Updated AC

Closing issue as PR got merged, no more work to do


Fail gracely on unsupported platforms

On windows 8.1 (not windows phone) the plugin fails with the following message:
   adding proxy for NfcPlugin
   WARNING: proximity device is null
   Registering for NDEF
   TypeError: Unable to get property 'subscribeForMessage' of undefined or null reference
   adding NFC Listener failed: TypeError: Unable to get property 'subscribeForMessage' of undefined or null reference
   Error in Error callbackId: NfcPlugin816380662 : adding NFC Listener failed: TypeError: Unable to get property 'subscribeForMessage' of undefined or null reference
   Exception calling native with command :: NfcPlugin :: registerNdef ::exception=adding NFC Listener failed: TypeError: Unable to get property 'subscribeForMessage' of undefined or null reference
   Initialized the NfcPlugin

-> better fail by testing for supported platforms

8.1 is working for me, can you provide your source code so we can replicate please?

Fixed in 0.6.6


Make regex methods public

Giving access to private functions makes it easier to use the existing
SwiftLexer as a template.

This pull request addresses two issues, one of which is a bug fix to the regex for multiline comments. I'm not sure how to separate the two commits, but you probably want to at least merge the multiline comment fix.

I will use your fork in my app. (I'll put credits)

I will use your fork. (I'll put credits)


Allow manual edit of date

Hi,
thanks for the great work. I'm using datetimepicker in combination with MopaBootstrapBundle in a Symfony2 project. I followed these instructions here to integrate it:
https://github.com/phiamo/MopaBootstrapBundle/blob/master/Resources/doc/form/4-form-components.md
I also read the documentation of datetimepicker back and forth but I can't find a way to allow the user to edit the date manually. Some users want to edit date or time manually because they are faster with it.
Everytime I change the input value it's set back to the last value after datetimepicker is closed. If I first close the picker and then edit the value nothing happens on blur and the old value is submitted in the form.
The datetimepicker demo shows readonly fields.
Is there a way to allow the user to override the value manually without using the datetimepicker?

same problem

How did you download this library?

I'm having the same problem.  Have you found a resolution yet?

@AuspeXeu Sorry it took me so long to answer, I'm now back into that topic.
As said in my initial description I'm using the library via the integration of MopaBootstrapBundle. Even after updating to the most recent version of this libarary I have the same behaviour. On the demo site unfortunately all input controls are read-only, so you're not able to test this use case. I prepared a jsfiddle for you:
https://jsfiddle.net/chmielot/umca5svy/2/
Steps to reproduce:

Click into the right side of the input field, so that the cursor stops behind the year 1980
Press backspace once, enter 8. Now you read 1988 as the year
Let the field lose focus by clicking somewhere else
The year changes back to 1980

Expected behaviour: Year stays as 1988 and is submitted like that with the form.
Can you please assist?

@AuspeXeu Can you at least tell me, if the library is even supposed to allow manual edit of dates and times? As said, the demo page doesn't even have one example that is not read-only.

@chmielot I am sorry, but I don't maintain the demo page. It was created by the initiator of this project. However, I am using the library for my own projects and the behaviour is fine there. I am not sure what the difference is here.

@AuspeXeu To be environment agnostic I prepared the jsfiddle. Can you either share the piece of code of your project so I can compare or could you have a quick look yourself?
It seems to me, that I'm not doing anything special in the fiddle, everything is plain standard, nothing fancy...
And you say that you can't reproduce the behaviour described for the fiddle in your own projects?

@chmielot that is exactly what I say, yes. I amusing the following links to include the library.
https://cdnjs.cloudflare.com/ajax/libs/smalot-bootstrap-datetimepicker/2.3.8/css/bootstrap-datetimepicker.min.css
https://cdnjs.cloudflare.com/ajax/libs/smalot-bootstrap-datetimepicker/2.3.8/js/bootstrap-datetimepicker.min.js
I know this is an older version than yours but when I put this version in you jsfiddle I have the same issues you describe. So I would assume it has to do something with the way you create/configure the datetimepicker. The code in my project looks like this:
HTML code
<input type="text" class="dtp" data-date-format="hh:ii dd.mm.yyyy">
and the relevant JS code
$('.dtp').datetimepicker({
  autoclose: true,
  todayBtn: true,
  todayHighlight: true
})
$('.dtp').datetimepicker('setDate', new Date())

@AuspeXeu Sorry for the long delay, I investigated even more. The main difference between your simple example and mine is that MopaBootstrapBundle uses a linked (mirror) field by default, to always submit a date in Y-m-d H:i:s format.
Unfortunately the datepicker component relies on the fact that the editable field always comes first in DOM tree, followed by the hidden (linked) field. This is due to a lot of locations doing:
element = this.element.find('input'); to find the editable field.
The fix for this is simple, I just changed the order of the fields:

    <input type="hidden" value="1980-01-13" id="pdb_promoterbundle_promotertype_birthday" name="pdb_promoterbundle_promotertype[birthday]" required="required">
    <input type="text" id="pdb_promoterbundle_promotertype_birthday_mopa_picker_display" name="" required="required" class="form-control" value="1980-01-13" />

to

    <input type="text" id="pdb_promoterbundle_promotertype_birthday_mopa_picker_display" name="" required="required" class="form-control" value="1980-01-13" />

    <input type="hidden" value="1980-01-13" id="pdb_promoterbundle_promotertype_birthday" name="pdb_promoterbundle_promotertype[birthday]" required="required">

This brought me closer to a solution but unfortunately there is one issue left.
I prepared a jsfiddle again with the newest library included: https://jsfiddle.net/chmielot/akapLr0p/
Working scenario (A):

Click between 8 and 0 of the year 1980 in the editable datetimepicker text input (note that the calendar opens, leave it open, just follow the steps)
Press backspace and change 8 to 9, the value becomes 1990.
Click on "Show values", you receive the expected result.

Non-working scenario (B):

Click between 8 and 0 of the year 1980 in the editable datetimepicker text input. The calendar pops up. Click in the exact same location again which makes the calendar disappear. Alternatively you can close the calendar popup by pressing escape.
Press backspace and change 8 to 9, the value becomes 1990.
Click on "Show values", you can see that the mirror field was not updated.

Scenario A works, because clicking outside of the calendar (by clicking on "Show values") calls clickedOutside which calls that.hide() which eventually calls setValue in line 433.  When the calendar is not visible anymore like in Scenario B, the values are not updated because hide exits early.
I thought about hooking into the calendar hide method, as everything works fine while the calendar is open. Please review my pull request and tell me if you need adjustments or if this is not the correct location for a fix. It shouldn't break anything else and seemed to be the most straightforward change.
Please find a working jsfiddle with the code changes applied here: https://jsfiddle.net/chmielot/vhf09gzm/
Feedback or merge is highly appreciated.

@AuspeXeu Kindly asking for feedback

@AuspeXeu I the proposed solution valid or do you want me to change something?


Conditionally validating within another condition doesn't work if value not present

I'm wondering whether there is a way to conditionally validate inside of another condition when the value is not present as currently it fails the validation if the value is not present in the JSON the schema is tested against. I can't show the exact code due to being internal to the workplace but here is some pseudo of what I'm trying to do:
// Rest of the schema and several other conditions beforehand all work fine. Key part is the bottom of this code
          allOf:
            - if:
                properties:
                  valueType:
                    type: string
                    enum:
                      - "someValue"
              then:
                required:
                  - valueWhichIsFailingIfNotPresent
                properties:
                  valueWhichIsFailingIfNotPresent:
                    $ref: '#/components/schemas/ComponentOne'
                allOf:
                  - if:
                      properties:
                        valueWhichIsFailingIfNotPresent:
                          minimum: 1
                    then:
                      $ref: '#/components/schemas/ComponentTwo' # The contents of this component is what states is required when validating if the is not present in the JSON. If valueWhichIsFailingIfNotPresent = 0 then it is not required and fine. If valueWhichIsFailingIfNotPresent > 0 it is required as expected too. Then if valueWhichIsFailingIfNotPresent is not present in the JSON, it states that these components' contents are required

Been trying to figure this out for a while but not sure it's possible now

Are you using a particular tool, or is this a general spec. related issue?

You need to add a requires keyword in your if clause. The way it is right now, the if clause will be true if the property doesn't exist, which means your then is evaluating when you don't want it to.

I think this question was answered and it's been inactive for a while so I'll close - @CharlieWoo01 let us know if you need anything else.


Fix incorrect date parsing

What problem does this PR solve?

Fix Incorrect date parsing in chunk and block encoding.
Ignores some tests not wanted when using tiflash ci.

Cause:
java.time.LocalDate should not be used, instead we should use org.joda.time.LocalDate.
MUST CHERRY-PICK WITH #1220

/run-all-tests