Comparison with population data, rework planetary vs interior mass/radius

Plots mass/radius/density throughout the model evolution on the mass vs radius diagram using planetary population data. Also plots this as density vs time, which will be particularly interesting for comparison with planets thought to be undergoing escape. The model downloads data from OSF to be able to do this. Closes #212.
Rework planet mass and radius variables. M_int now represents all dry mass enclosed in the interior (core+mantle), not including volatiles. M_planet = M_int + M_atm is the total planet mass. R_int represents the radius of the atmosphere-mantle interface, and is also what the user provides in the config file. R_planet no longer exists, but the best analogue for this is z_obs+R_int. We can think about how to rework the mass calculations in the future, but this is already somewhat better than treating M_planet and M_int as if they're the same thing. Closes #213.
Since the new variable adds an extra key in the output csv file, I had to update the dummy test data, but we can see that it's getting the exact same results. I've attached the old T_surf and new T_surf columns to prove this. This PR comes with a lot of file changes because I updated the examples and renamed variables throughout the code. Feel free to ignore the updated examples, and you can see that most of the changes are find+replace. You can filter to just viewing changes to py files, if that helps.
Other minor things:

Fixed incorrect axis limits on interior plot
Make sure that rounding errors don't cause problems with CALLIOPE outgassing. Sometimes the elemental mass would be ~1e-34, which is difficult for calliope to solve in some instances. When the mass is small, it is now rounded to zero, which fixes the problem.

I don't know why the test is failing now. It seems to be due to some issue reading the configuration file?
The test runs fine when I execute the same commands in the Python REPL, and also works fine when I run proteus start --config input/dummy.toml.

Do you want me to review this now, or do you first want to figure out why the tests are failing?

It would be great if you could review it now, and check from a physical perspective if these changes make sense.
I think we can potentially discuss this configuration issue with @stefsmeets at the next PROTEUS meeting.

Turns out that the reason it is failing is not because of the integration test. It's when the tests check that the toml files in examples/ for validity that it fails, because the fields which are converted from "none" to None do not get saved.
See this issue: https://github.com/uiri/toml/issues/346

Thanks Tim!

Turns out that the reason it is failing is not because of the integration test. It's when the tests check the toml files in examples/ for validity that it fails, because the fields which are converted from "none" to None do not get saved. See this issue for a description of a similar problem: uiri/toml#346

This is a somewhat awkward side-effect from toml not recognizing a 'null' value: https://github.com/toml-lang/toml/issues/30
I think copying is also the way to go for the moment, depending on how much we lean on converters in the future. TOML really seems to be made as a one-way format (toml -> dict). If we want to go back (dict -> toml), the alternative is having a custom TomlEncoder to go from None -> 'none'.


Issues with Dipy MAPMRI Outputs

Summary
The MAPMRIOutputSpec has namespace reserved for several MAP-MRI metrics that dipy computes (NG, Parallel NG, Perpendicular NG, and general coefficients) that it doesn't actually provide an output for. It appears that it attempts to save the coeffs inqsirecon/qsirecon/interfaces/dipy.py (see below) but the method is called incorrectly and it just doesn't provide an output.
https://github.com/PennLINC/qsirecon/blob/09951564efebab13542df87e446f1cf71da35d08/qsirecon/interfaces/dipy.py#L294-L295
Additional details

QSIRecon version:
Docker version:
Singularity version:

What were you trying to do?
What did you expect to happen?
What actually happened?
Reproducing the bug

Hey Taylor! What exactly do you need?

It looks like the ng, perng, and parng files can't be generated if anisotropic_scaling is set to false in the MAPMRI_reconstruction node of the reconstruction workflow, and none of the built-in ones have it set to true. If you have a reconstruction workflow that does have it set to true, I was hoping you could test my branch locally and see if the results look good to you. Otherwise, I can modify one of the existing tests, but I don't know how anisotropic_scaling would affect the results.

@tsalo I just double-checked the MAPMRIInputspec, it seems like it's set to default to anisotropic_scaling=True

I think all of the built-in workflows (dipy_mapmri.json and multishell_scalarfest.json) explicitly set it to False though.

I'll grab your branch and give it a test as soon as I can, I'm confident it'll run. Is there a reason that we know of that the built-in from qsirecon would have defualts that differ from the dipy defaults? It's 100% set to true by default from dipy proper https://github.com/dipy/dipy/blob/1cc364f2b20479b29063dd7ed231d9ab3fcd710f/dipy/reconst/mapmri.py#L90

That's a question for @mattcieslak. He told me his reasons on Slack, but they went over my head.

There was a paper awhile ago that showed that (IIRC) anisotropic scaling was bad for tractography but good for estimating propagator measures. Back when I wrote these workflows I was using the MAPMRI fits to estimate ODFs for tractography so I had turned it off.

Should either the dipy_mapmri.json and multishell_scalarfest.json be set to dipy defaults or should at least MAPMRIInputSpec be changed to be consistent then?

I think definitely for multishell_scalarfest.json we should change it to the dipy default. For dipy_mapmri.json we can add a note to the documentation to change this parameter depending on your goals.

I didn't change the recon configs, so that'll have to happen in a separate PR.


supportsCredentials issues

Hey there,
thanks for your plugin!
I 'm trying to use it, in order to allow CORS requests between a client(https://something.app) and an API(https://api1.something.com).
While GET and POST requests work, I 'm now trying to make Authentication work between the client and the API. I thought I was on a dead end when I read that there is no cookie sharing with CORS. But then I noticed supportsCredentials flag inside the cors config and thought I should try it.
Unfortunately I 'm getting the error

XMLHttpRequest cannot load https://api1.something.com/v0/customer/phone. A wildcard '*' cannot be used in the 'Access-Control-Allow-Origin' header when the credentials flag is true. Origin 'https://something.app' is therefore not allowed access. The credentials mode of an XMLHttpRequest is controlled by the withCredentials attribute.

But the thing is, I don't have a wildcard for allowedOrigins!
My config/cors.php is:
    'supportsCredentials' => true,
    'allowedOrigins' => ["https://something.app", "https://www1.something.com"],
    'allowedHeaders' => ['*'],
    'allowedMethods' => ['GET', 'POST', 'PUT',  'DELETE'],
    'exposedHeaders' => [],
    'maxAge' => 0,
    'hosts' => []

I tried clearing caches and resubmitting the conf through the vendor:publish command, but no luck :/
Am I missing something?
Thanks for your time!

My bad.
I guess I should always check apache configuration before opening a new issue.
Sorry!

@mitsest  what was the problem with your apache configuration? I'm facing similar issues and can't track the problem down.

@molerat619 can't remember, sorry.  Maybe, CORS was disabled on the apache side (Header set Access-Control-Allow-Origin "*")?


Issue in populateDB.js error in canteen schema

whenever script is started it shows accounttype not defined error

That seems like a good bug , I am assigning this issue to you , raise pr after 7:00 pm


Redis Docarray with Sentinel Client

I use the  Redis Operator in Kubernetes.
Sadly I could not figure out how create an DocumenArray via Redis if i just have a connection via a Sentinel.
Normally I create some clients like this:
                sentinel_pool = Sentinel(
                    [(client["host"], client["port"])],
                    socket_timeout=client["timeout"],
                    health_check_interval=280,
                    password=client["password"],
                    socket_keepalive=True,
                    decode_responses=auto_decode,
                )

                # Master Client
                master = sentinel_pool.master_for(
                    "mymaster", socket_timeout=client["timeout"]
                )

                slave = sentinel_pool.slave_for(
                    "mymaster",
                    socket_timeout=client["timeout"],
                )  # readonly

but as i see it the client built in docarray has no settings for a connection pool from Sentinel.
Am i missing something crucial here?

I just mad a variable and provided the Sentinel Client directly skipping the construction of a new one completely. This seems to work.


feat(foundation): support passing env vars to nested stacks (kinda)

[Finishes #184993937]
I'm not thrilled by this solution, but CloudFormation makes it inconvenient to pass arbitrary values to a nested stack and (as far as  I can tell) all but impossible do do anything with them.
This lets us use Fn::ToJsonString to pass an arbitrary map to the nested stack and then that map (as a string) is injected into the Lambda as an environment variable. At initialization, the lambda JSON decodes the map and sets each value on the environment

Codecov Report

Merging #82 (ba7a499) into main (77885c5) will increase coverage by 13.61%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #82       +/-   ##
===========================================
+ Coverage   49.80%   63.41%   +13.61%
===========================================
  Files          25       15       -10
  Lines        1510      738      -772
  Branches      349      178      -171
===========================================
- Hits          752      468      -284
+ Misses        754      265      -489
- Partials        4        5        +1

see 17 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Retag and push fails by "Retrying in" -- Not using pre-existing layers

Description: Retag and push fails by "Retrying in" -- Not using pre-existing layers
I locally retagged an official debian image as rsvp/jessie and pushed it to docker.io as my public repo, however, that failed in "Retrying in ..." as it tried to actually upload the bulk. All the necessary layers should already be pre-existing at Docker's hub, hence expected behavior would be no uploading necessary.
Steps to reproduce the issue:

Pull any known image at Docker's hub
Retag this image as USER/NEWNAME
Push that retagged local image to hub

Describe the results you received:

Observed failure to find pre-existing layers
Observed upload fail by numerous "Retrying" messages.

Describe the results you expected:

Struggle to upload should not even occur since the necessary layers pre-exist at the hub repository

Additional information you deem important (e.g. issue happens only occasionally):

Major problem especially for popular large images, e.g. continuumio/anaconda
Not suspecting a permission problem since operations are performed in public space

Output of docker version:
Docker version 1.12.2, build bb80604

Output of docker info:
Containers: 18
 Running: 1
 Paused: 0
 Stopped: 17
Images: 54
Server Version: 1.12.2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 62
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host null bridge overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-43-generic
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 11.64 GiB
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/

i think cross-repo sharing of layers depends on the registry version in use; it's possible Docker Hub does not use that version yet.
/cc @dmcg @tonistiigi

Is the following info posted by  @stevvooe on May 14, 2015 for #13210 -- more than 17 months ago, still valid?

Efficiency-wise, the V2 protocol is biased towards pull rather than push. The behavior you've described is a side-effect of access-control in the V2 protocol. From a high-level, the image has a list of blobs that make it up. Currently, all of these blobs need to be "mounted" into a remote repository (ie "centos"). When pushing these blobs (layers) for the first time into a new repository, the client must "prove" they have access to the blob by sending it. This gets verified and the blob then becomes available in the new repository. Even though data is sent "unnecessarily", data is only ever stored once.

If still current, there must be a better way to furnish proof than to resend a blob which could be enormous in size. It might seem reasonable for small private repos, however, if the blobs are public, then the hash should be sufficient proof to prevent unnecessary upload of a pre-existing layer.
The "Retrying in ..." upload issue can possibly be averted by setting --max-concurrent-uploads (in the daemon dockerd) but hinges on the individual user's internet capacity.
Would be grateful to see other alternative solutions. Thanks!

@rsvp I think @thaJeztah was pointing that out: we do have a better way of proving ownership, called cross-repo mount, which may not be implemented by the hub.

It might seem reasonable for small private repos, however, if the blobs are public, then the hash should be sufficient proof to prevent unnecessary upload of a pre-existing layer.

So, now the hashes of your content become sufficient to access that content? If you leak one of your layer hashes, do you want everyone else to have access to that data now? Even telling an attacker that a blob is public or private leaks too much information here.
I'd recommend you contact support.docker.com and discuss with them.

Re: Push to registry v2 does not detect already pushed layers #1411 @RichardScothern commented on Feb 2, 2016:

The v2 security model necessitated images being re-uploaded. Registry 2.3 does not require a re-upload if the client has read access to an existing layer in another repository.

Expected behavior would be that if the repository is public, then the read access is obviously set for everyone -- thus re-uploading would NOT be necessary. This certainly would make sense for saving disk space at the hub. And presumbly there's no security risk from exposing the hash values of the public blobs involved (assuming OP was not careless).
Q: Does docker.io use cross-repo mount to prove ownership? How does an user query a hub about its features from CLI, e.g. the Registry protocol version?  Appreciate your comments -- thanks!

Expected behavior would be that if the repository is public, then the read access is obviously set for everyone -- thus re-uploading would NOT be necessary. This certainly would make sense for saving disk space at the hub. And presumbly there's no security risk from exposing the hash values of the public blobs involved (assuming OP was not careless).

To be clear, we don't store the same data twice. We only force clients to prove they have the data if they want to redistribute it, unless they have permissions to mount from another repository.
While it may seem "obvious", the issues become clear when you start analyzing how to implement the ACLs in such a system. ACLs are done at the repository level to keep the size of ACL from bloating as the number of blobs grow and to avoid having to maintain an extra service (database or otherwise) to maintain this. There are also complexities involved when two entities disagree on the ACL of a blob. Let's say Bob says a blob must be private but Alice insists that it is public. Do you just serve up that data to everyone because Alice said so?
To deal with this, there is no such thing as "public" repositories, from the registry perspective. Every user authenticates access to a repository, and that token is presented with each API request. When a repository is "public", this token is passed out to whoever asks, but it is still authenticated as anonymous.

Q: Does docker.io use cross-repo mount to prove ownership?

Again, this is a question for support.docker.com. docker.io is separate from the docker project and associated protocols.

How does an user query a hub about its features from CLI, e.g. the Registry protocol version? Appreciate your comments -- thanks!

The mount is tried, optimistically,  here. If it works, then no upload is necessary. If not, the upload proceeds normally.

@stevvooe Thanks very much for your thorough response. I think your Alice/Bob scenario nails the current issue. Very interesting that: there is no such thing as "public" repositories from the registry perspective -- this non-obvious fact should help resolve any misunderstanding regarding ACL (Access Control List), esp. why Docker may not be using pre-existing layers during a push to a hub.
As for the security aspect, Bob must trust Alice to keep his blobs private -- much like how his decrypted GPG private text should not be posted on her public blog for everyone's reading pleasure.
SUMMARY: This issue began with the intention to archive an official image, which was public, to an user's repository in a non-local hub for the sake of future replication, lest the official image be deleted. The ACL as described above indicates that necessary blobs will be accessible by the user even if official owner deletes their (pointers to the) blobs. The cost involved is the download time involved in the push operation, even if one knows the layers already exist at the hub -- to prove actual ownership.

@rvsp Great summary!
I would like to add that this is addressed in registries with cross-repo mount, avoiding the extra upload cost, if supported.
Let me know if you have more questions!


Fix and normalize viewBox size

This would be awesome! 🎉  It also helped me figure out how to get my sprites to scale - I had to add viewBox="0 0 8 8" before the CSS sizing would take effect.

I don't know though if @somerandomdude is still around to merge that 😕

Just fyi, it's better to switch to an open-source and maintained library such as Github Octicons.


Rename Eco-advices on the main page

On the main page rename Eco-advices title as Tps & Tricks.
Acceptance criteria:
On the main page, the title for the advice section is Tps & Tricks.
#280

resolved


azure-devops: fix url connection string

Describe the behavior changes introduced in this PR
Linked Issues?
resolves #724
w/ fix:
11-18-2022 17:07:17 INFO     Loading CommittimeTypeConfig, inputs below:
11-18-2022 17:07:17 INFO     provider='git', from env var PROVIDER
11-18-2022 17:07:17 INFO     Loading GitCommittimeConfig, inputs below:
11-18-2022 17:07:17 INFO     username='whayutin', from env var API_USER
11-18-2022 17:07:17 INFO     token=REDACTED, from env var TOKEN
11-18-2022 17:07:17 INFO     namespaces='mongo-persistent,basic-python-tekton', from env var NAMESPACES
11-18-2022 17:07:17 INFO     git_api='https://dev.azure.com/whayutin', from env var GIT_API
11-18-2022 17:07:17 INFO     git_provider='azure-devops', from env var GIT_PROVIDER
11-18-2022 17:07:17 INFO     tls_verify=True, default value; TLS_VERIFY was not set
11-18-2022 17:07:17 INFO     Watching namespaces: {'mongo-persistent', 'basic-python-tekton'}
/home/whayutin/OPENSHIFT/git/KONVEYOR/PELORUS/upstream/pelorus/.venv/lib64/python3.10/site-packages/urllib3/connectionpool.py:1045: InsecureRequestWarning: Unverified HTTPS request is being made to host 'api.cluster-wdh11092022b.wdh11092022b.mg.dog8code.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
11-18-2022 17:07:18 INFO     metric.commit_time 2022-11-11 19:59:51+00:00
/home/whayutin/OPENSHIFT/git/KONVEYOR/PELORUS/upstream/pelorus/.venv/lib64/python3.10/site-packages/urllib3/connectionpool.py:1045: InsecureRequestWarning: Unverified HTTPS request is being made to host 'api.cluster-wdh11092022b.wdh11092022b.mg.dog8code.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
11-18-2022 17:07:18 INFO     Collected commit_timestamp{ namespace=mongo-persistent, app=todolist, commit=247e9859fb232f2b59b17391c7402814d82651d9, image_sha=sha256:04559c4ebdde446ade9a67cd58124fdcb9e28f2ffc7fac923e0f3845c9f3d54a }<PHONE_NUMBER>.0

/retest

@KevinMGranger all the tests passed, only one dco is not, but I think we could merge this if you are happy with implementation?

process_max_fds 1.048576e+06
# HELP commit_timestamp Commit timestamp
# TYPE commit_timestamp gauge
commit_timestamp{app="todolist",commit="247e9859fb232f2b59b17391c7402814d82651d9",image_sha="sha256:04559c4ebdde446ade9a67cd58124fdcb9e28f2ffc7fac923e0f3845c9f3d54a",namespace="mongo-persistent"} 1.668196791e+09

/retest 4.11-e2e-openshift

/test  4.11-e2e-openshift


Teste

          1 Message

  :book:
  @cs-bruno-rendeiro is not a contributor yet

Current coverage for DangerPoc.app is 42%
✅ No files affecting coverage found

Powered by xcov

  Generated by :no_entry_sign: danger


fix(license): Updating LICENSE

:tada: This PR is included in version 5.0.1 :tada:
Your semantic-release bot :package::rocket:


skid

Everything stolen from: https://github.com/Smug246/Luna-Grabber

Everyone is doing it why u finding me
6 Haz 2023 Sal 02:59 tarihinde Blank @.***> şunu yazdı:

Everything stolen from: https://github.com/Smug246/Luna-Grabber
—
Reply to this email directly, view it on GitHub
https://github.com/vasillax12/-p123/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAG6ENISVE3MFU7QXIP3FI3XJZXG3ANCNFSM6AAAAAAY3ULL3E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

this is opened source code which means i can do what ever i want with it
maybe they should try obsufacete?

6 Haz 2023 Sal 07:23 tarihinde Alp Yamaner @.***> şunu yazdı:

Everyone is doing it why u finding me
6 Haz 2023 Sal 02:59 tarihinde Blank @.***> şunu
yazdı:

Everything stolen from: https://github.com/Smug246/Luna-Grabber
—
Reply to this email directly, view it on GitHub
https://github.com/vasillax12/-p123/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAG6ENISVE3MFU7QXIP3FI3XJZXG3ANCNFSM6AAAAAAY3ULL3E
.
 I just came across your repo when I was surfing.

this is opened source code which means i can do what ever i want with it
maybe they should try obsufacete?

If the owner obfuscate it, they would lost trust of the community. You cannot just steal other's hard work and frame that as yours. If you still insist that you can do anything with it that you want, so fine, I can do anything that I want too.


Cannot or really hard to scroll the map when put it in SingleChildScrollView

My app need to add the map inside a list of views and put altogether in a SingleChildScrollView. The map displays well but really hard or cannot scroll or zoom the map.

Let me check this issue and see if it resolves the problem. I'll get back to you once I've had a chance to test it out. Thanks @andj207


Fixing JEP link

Both the link and the link text were incorrect, but in different ways. :grin:

too slow :)  #5

JEP-229 not JEP-221

JEP-229 not JEP-221

I am confused - this is a Jenkins Plugin not a Jenkins component?


figure out how to keep the clear filter 'x' with the narrow select box on evidence grid cols

Done.


Can't use "Enter" key to accept an IntelliSense item

monaco-editor version: 0.11 on the website I'm assuming
Browser: Version 64.0.3282.186 (Official Build) (64-bit)
OS: Windows 10
Steps or JS usage snippet reproducing the issue:

Open the playground website.
Hit Ctrl+Space
Hit Enter
A newline is inserted instead of the proposal.

I feel like it wasn't like this before...

Thanks for fixing this so quickly, @alexandrudima. Will you be publishing an update to npm?

Yes, I just have a couple more things to do

Great to hear! Thank you and everyone else for the new release!

Just published 0.11.1

Things look good in the Monaco Editor Playground. Thanks, Alexandru! :)


Fix a few README typos

Bumped into a few typos in the README while working on my list of plugins for IDA. Most importantly the name of the project was botched in the first line so here's a PR to correct it.

Thanks for this, I was too tired when I wrote the readme :)


[REQUEST] Add support for Torrent-Igruha games and repacks

Describe the solution you'd like
Considering that https://itorrents-igruha.org/ has a shit ton of games it would be great to see these repacks get added to all of the other ones, that's it.

site with stillers in repacks. not reccomend.

лучше https://byruthub.org/

No!!!! It's got viruses, watch YouTube.


feat: add exclusion list for ActorMiddleware

Description
Introduced ability to create ActorMiddleware with exclusion of specific functions. This is commonly used when implementing HealthCheck in functions.
References

#0000

Codecov Report

Merging #108 (23585db) into main (b2c125f) will decrease coverage by 0.78%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   67.60%   66.82%   -0.79%
==========================================
  Files         136      136
  Lines        2482     2511      +29
  Branches      361      364       +3
==========================================
  Hits         1678     1678
- Misses        684      713      +29
  Partials      120      120

Flag
Coverage Δ

appcommon
18.06% <0.00%> (-2.65%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...p/source/FunctionApp/Middleware/ActorMiddleware.cs
0.00% <0.00%> (ø)

...ource/FunctionApp/Middleware/JwtTokenMiddleware.cs
0.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


allow specifying more settings in service labels

Most specifically client, server, tunnel timeouts, because some of my services can be working up to 5 minutes per http request. Others use websockets.

Do you have a snippet of an existing configuration you'd like to accomplish? The amount of config options and combinations supported by HAProxy is almost unlimited. An example of your use case would help me focus on you immediate needs.

Also, it would be helpful if you could provide a set of commands that would validate this feature. Do you have a public service that could be used for testing?

For me currently it's probably only timeout server and timeout tunnel. It's not really public, I emailed you the address.

The option to change the default value of timeout server already existed in the proxy. I just added the option to specify timeout tunnel as well. It is available as the release 1.154.
Both settings can be changed through environment variables TIMEOUT_SERVER and TIMEOUT_TUNNEL. See the config documentation for more please.
Please don't hesitate to contact me if it does not work as expected or you need more info.

I'm very sorry. I absolutely wasn't clear. If I specify TIMEOUT_SERVER this applies to ALL my services, and I meant support for something like a com.df.timeout_server label, so it could be applied per-service.

Please take a look at the release 1.156 and let me know if that's what you were looking for. If it is, I would appreciate if you could try it out and let me know if it works as expected.

Yes this work as expected. Thanks. (remark: the generated configfile misses a trailing LF).
I never used haproxy before, but indeed it has a lot of options. Creating a universal auto-reconfiguring proxy will be challenging, using service labels only. On the other hand, maybe there are only two most-used cases: (1) proxying multiple backend services using virtual hostnames for each, and (2) proxying multiple backends under multiple basepaths.
Anything more complicated will probably need a custom written config file.

Thanks for trailing LF note. I added it to the code (it'll be released in a few minutes).
I think that the project has many more combinations than those two. It will continue growing to accommodate many additional use cases. Feel free to create an issue if you have a specific use case and I'll add it as long as that would not hurt other users. In the meantime, the existing combinations can be easily extended by providing custom templates. The only downside is that using templates requires knowledge of HAProxy config.


Undo move to pep440_rs 0.4.0.

pep508_rs 0.2.4 (which transitively depended on pep440_rs 0.4.0) was yanked, but poc-monotrail wasn't updated. So now things fail to build in a different way.

Maybe we can just bump it upstream (https://github.com/konstin/poc-monotrail/pull/65)


add cross-compile target config

This is necessary for compiling krustlet-wasi and krustlet-wascc for 64-bit ARM devices (and eventually 32-bit ARM as well, though that is currently not supported).
TODO:

[ ] add documentation for cross-compiling

Signed-off-by: Matthew Fisher<EMAIL_ADDRESS>

relies on #266.

will add docs as a follow-up.


Add oembed

I did some work around this. Going to close it out for now until we come up with more actionable TODO.


Need screenshots or additional detail

This page is not clear on where or how to get to the Admin Setting to create the connection.

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

ID: eab7955b-7cac-cb2a-c115-ae3036feb244
Version Independent ID: eab7955b-7cac-cb2a-c115-ae3036feb244
Content: Invoice capture solution advanced settings - Finance | Dynamics 365
Content Source: articles/finance/accounts-payable/invoice-capture-advanced-settings.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @sunfzam
Microsoft Alias: zezhangzhao

Bug 759448 has been logged to address this issue.

Go to Admin system -> Default solution, click “New” and select “Connection Reference” again. On the Connection creation page, in the Connection type field, select Office 365 Outlook.


Kotlin 1.5 updates

Also threw in watchosX64 target for good measure

When will there be a new release including these changes?

@RdeWilde A release went out today! Version 0.1.9


Cannot install java with brew cask

Running brew cask install java I got an error

Error: Download failed on Cask 'java' with message: Download failed: http://download.oracle.com/otn-pub/java/jdk/10+46/76eac37278c24557a3c4199677f19b62/jdk-10_osx-x64_bin.dmg

Oracle download links for all versions of Java seem to be broken at the moment.
Trying to download from the Oracle website will also 404.

Seems like downloads are working again.

@ggsp yes, I will close this issue.


Request: Add Ability to Reset SerializerBuilder

The Problem
For certain test cases, we wish to use certain serializer settings (particularly using IgnoreMember(Type declaringType, string name) ). But it would be nice if these did not persist to other test cases.
We have implemented scrubbers in situations where that is appropriate (e.g., range and/or formatting validation). But for certain cases where we are only verifying something very targeted, being able to use custom serializer settings per test run would be helpful.
Potential Solution(s)
Regardless, we would need to add a public static void Reset() method onto SerializerBuilder.
The simplest solution would be to have the reset function just reset all private fields and properties on the static class to the default values.
Or, could move the storage of settings overrides to some private singleton class and the reset could be done by simply performing settingsStorage = new SerializerBuilderSettingsStorage() and have the constructor do the appropriate initialization.
Either way, if you would be willing to accept a change like this I could make a pull request with one of these solutions.
Thank you for your time.

there is an overload of VerifyWithJson that accepts both a JsonSerializerSettings and a scrubber. would that be sufficient for your requirements?

@paulsavides @jvonruden thoughts one my comment?

@SimonCropp, apologies for the delay.
If we were to use the override that accepted the JsonSerializerSettings we would need to rewrite most of the logic that the SerializerBuilder class is doing. As in, we want to use what that class provides more dynamically; the override doesn't really get us further to that goal.

ok can you submit a PR with what you have in mind

@paulsavides does this work for you https://github.com/SimonCropp/ObjectApproval#scoped-settings  ?


set minimum price to 0.01

[x] Testing implications of lower minimum OCEAN price
[ ] custom cap (to be defined)

Hey @kremalicious , though I'll just shoot you a comment here. Thanks for the Rinkeby tokens.
I've tried to create the pricing on the pool, however it looks that I'm unable to mint the amount of Data Tokens that I'm looking for. I'll provide upwards of $OCEAN 5000+ initial liquidity at an IDO price of 0.02 - 0.05. So the amount of Data Tokens available will be between 50K - 120K. Is it possible to change this, so I'm able to mint that much?
I think the total supply is currently capped at 1000.
Let me know if you can help us for this.

Change in this PR would be totally fine so far.
But the datatoken cap indeed is hardcoded right now, or more precisely the market does not capture or pass it, making ocean.js fall back to 1000. The tricky part here is that the cap would need to be set right in the publish form and not during price creation as creating the metadata also mints the initial data tokens. And it's hard to make majority of users understand this when they just create metadata and don't think about pricing yet. Which is why we were holding off on just adding it as a new field in the UI in publish form. The ideal solution here would be to split up metadata publishing & datatoken creation, which we already have in mind but no ETA. On the other hand in v4 this all seems to be not an issue anymore.
Some options I can think of to make your use case possible in a timely way where we can pick one:

Hardcoded custom cap which we pass to ocean.js during publish. So add to this PR, and you would use its preview link for publishing your assets. Quickest solution for everybody but you would have to live with the same cap on every asset.
Expose cap as a new optional input field in publish form for everybody. This might get feasible already to be put behind a new collapse-by-default Advanced Settings area in publish form introduced in #707. So this is not as quick as 1 as we need to finalize #707 first and then see more UI & copy work will be required.
Wait for v4, but most likely not an option for you.

Hi @kremalicious ,
thanks a lot for presenting all the options. I’m inclined towards Option 1, as it seems to be the most timely solution. Also I feel Option 2 will be obsolete when v4 launches and since the micro-cap IDO is a trial & error, I don’t want to create too much work for your team.
Before going ahead, could you please help to clarify the below for my understanding?

will the hardcoded custom cap be linked to my publishing address? Say, I settle on a supply of 250K for this IDO, then for any future IDOs, this would be my personal maximum supply cap correct?

thanks again for helping and supporting 🙏

Thanks, just tried with the vercel link, mint still exceeds max cap on Rinkeby

Well you have to publish a new asset, only that will have the cap hardcoded in here. The cap is set during the datatoken creation which is happening before the price creation. So you can't set it during price creation because the datatoken is already minted. Price creation is creating the pool (or fixed rate exchange), not the datatoken itself.
So nothing in the DDO regarding the cap there will be affected by anything you do on that existing asset https://market-git-feature-micro-cap-oceanprotocol.vercel.app/asset/did:op:81DC6629b05F43D522049382a013EB7E2772cC99

Working! Thank you :-)

Just created the price & supply of 100'000 Data Tokens! Exciting!
@kremalicious , please advise how do we go about the custom ticker ?

please advise how do we go about the custom ticker

to allow this in the UI would be a bit more complex to setup, hardcoding could work but also needs quick investigation as we auto-generate a new one on initial render of form. But we are not going to update it for you for every planned IDO anyway. As I said, ideally you would create the initial asset programmatically which you seem to have done in the past already and would give you all the flexibility you need. Once it's published you can then just pick it up and continue in the UI normally to create the price.

Hi @kremalicious , appreciate the offer to hardcode it for this one. Once it's done, will I have to create it via the link or will it show up immediately? If this is the case, let's wait until I'm fully ready with the dataset in about 15 days or so.
Thanks

Revisit after v4


truncate any array or string field to a specified length

I want to truncate any array or string field to a specified length when serializing, how can I do this?
I use for preview logs and don't need full JSON.

I successfully registered the string encoder, but I didn't find how to register any array encoder.

I solved it by decorating the encoder and copying the slice header.


Refactor bids() function

The process of working on #100 led to a lot of changes, not all of which are related. I thus thought it might be easier if I release the changes across a few PRs based on content theme.
The main goal here was for bids() to return Paths rather than strings. Snakemake can consume Paths as input/outputs just fine, and Paths are just more convenient.
Along the way, I refactored a few things:

Form filename and folder in a "one-liner" list creation, rather than gradually appending items. This makes the code much more concise and clear.

Add type annotations

@akhanf, am I good to merge this in?


Add auto-complete to flow/global typedInput types

This is still a work-in-progress, but most of the required functionality is there.
This adds auto-complete to the flow/global typedInputs.
A new flag has been added to the /context api endpoints - keysOnly that tells it to only return the keys at the matching context level, rather than the full contents. I'm not 100% happy with this flag-based way of changing the response of the api, but it'll do for now.
Remaining tasks:

Each typedInput caches its requests independently. This needs to be improved to cache across the editor - for example, a single edit dialog with multiple typedInputs should use the same cache. The reason I haven't started with a global cache in the editor is that we need a way to refresh the contents of the cache so it is up to date.
Need to handle keys that require ["bracket notation"] rather than dot.notation to form a valid property expression.

Just pushed an update to support bracket notation in the completions:

Last remaining piece is to improving the caching


Add WSS support using recently added class SecureServer from React.

Great work, I'm standing by having this merged!

Only a question. Why
$sslconf = array()
and not
$sslconf = NULL
and why
if (!empty($sslconf))
and short
if ($sslconf)
? :)

@Daijobou these are indeed possible other ways of doing this, but as far as is visible from the code, there is no real "dominating" reason to do this (apart from code readability). There are (as far as I know in any case) no security bugs that are introduced by using empty(...) and array(...). Although I like the approach with is_array(...) more, since this also ensures the specified object is indeed an array.
Thanks :)

Thanks for the suggestions. I already did the corrections.
Example usage:
`$server = SecureIoServer::factory(
new HttpServer(
new WsServer(
new Socket()
)
),
3334,
'<IP_ADDRESS>',
[
'local_cert' => '/tmp/certificate.pem',
'allow_self_signed' => true,
'verify_peer' => false
]
);

Good job! You may have just made one of the most important changes for me,
determining how much I will use this repository from now on. I am more than
convinced I am not the only one.
On Mon, Mar 13, 2017 at 1:30 PM, Eliseu dos Santos<EMAIL_ADDRESS>
wrote:

Thanks for the suggestions. I already did the corrections.
Example usage:
`$server = SecureIoServer::factory(
new HttpServer(
new WsServer(
new Socket()
)
),
3334,
'<IP_ADDRESS>',
[
'local_cert' => '/tmp/certificate.pem',
'allow_self_signed' => true,
'verify_peer' => false
]
);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ratchetphp/Ratchet/pull/500#issuecomment-286069654,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF9Y9LZlkj_c6L5V7iGdLZrZY45AKJKzks5rlRq3gaJpZM4MHQMv
.

--
Samuel Allan
<EMAIL_ADDRESS>Work Email
<EMAIL_ADDRESS>

Needing so to speak I use this library quite a lot and in production on 20 clients.
The 'continuous-integration' tests failed and when I checked the logs for what I saw is their error and not the code .. how to proceed?

Thanks for the contribution. I will review this shortly. The CI check is failing because Travis is using a newer version of PHPUnit with PHP7 boxes where Ratchet's test suite is using a class that was removed.

This has been merged into the 0.4 branch

In Ratchet/App.php is Support for SSL still missing?
https://github.com/ratchetphp/Ratchet/blob/0.4/src/Ratchet/App.php#L79
App is used in example for version 0.4
https://github.com/ratchetphp/Ratchet/tree/0.4

Maybe I'm missing something but shouldn't it be also in constructor, not factory method only?

@Daijobou I've updated App to include a parameter for SSL Context options
@simPod If you use __construct you would created the React SecureServer yourself instead of letting IoServer do it for you, like in Factory.
I've also updated the ApiDoc on the function
(These changes I speak of are in the 0.4 branch)

Quick update: I haven't been able to test this and it's the only thing holding up a new release. I've temporarily removed it from release 0.4. I'll put it back in for 0.4.1 when I'm able to test. In the mean time you should still be able to support SSL with Ratchet by creating the Secure React Socket and injecting it into the IoServer constructor.

Hello, I'm trying to use the React SecureServer with Ratchet IoServer. I am getting a fatal error:
Catchable fatal error: Argument 1 passed to React\Socket\SecureServer::__construct() must be an instance of React\Socket\Server, instance of Ratchet\Http\HttpServer given.
Part where it goes wrong:
$webServer = new Ratchet\Server\IoServer(
new React\Socket\SecureServer(
	new Ratchet\Http\HttpServer(
		new Ratchet\WebSocket\WsServer(
			new Ratchet\Wamp\WampServer(
				$pusher
			)
		)
	)
),
$webSock,
array(
    'local_cert' => '/var/cpanel/ssl/apache_tls/demonstratie.alfanet.be/combined.pem',
    'allow_self_signed' => true,
    'verify_peer' => false
)
);

Am I doing something stupid? :p

I have made all the changes you have suggested in this thread, I always get the same error from the websocket: ERR_SSL_VERSION_OR_CIPHER_MISMATCH

Good Morning .... I'm reusing the lib in a new project again. and the solution for using WSS is working perfectly. The only errors were in relation to the certificate itself, but already correct.

do you know which certificate i've to use?
I put the certificates .crt I received from digicert like this:
`<?php
use Ratchet\Server\IoServer;
use Ratchet\Http\HttpServer;
use Ratchet\WebSocket\WsServer;
use App\Http\Controllers\ChatController;
use React\Socket\SecureServer as SecureReactor;
require  'vendor/autoload.php';
$server = IoServer::factory(
new HttpServer(
new WsServer(
new ChatController()
)
),
8080,'<IP_ADDRESS>',
[
'local_cert'        => '/certs/da_mod.pem', // path to your cert
'local_pk'          => '/certs/chiave.key', // path to your server private key
]
);
$server->run();
?>`

just the .pem

@cboden  @EliseuSantos do you know why on my android app (is a web App) the socket is on CLOSED STATE?
on iOS and chrome(pc/mac) it work without problems
my server php file
`<?php
use Ratchet\Server\IoServer;
use Ratchet\Http\HttpServer;
use Ratchet\WebSocket\WsServer;
use App\Http\Controllers\ChatController;
require  'vendor/autoload.php';
$server = IoServer::factory(
new HttpServer(
new WsServer(
new ChatController()
)
),
8081,'<IP_ADDRESS>',
[
'local_cert'        => '/var/www/laravel/certs/da_mod.pem', // path to your cert
'local_pk'          => '/var/www/laravel/certs/chiave.key', // path to your server private key
'allow_self_signed' => true,
'verify_peer' => false
]
);
$server->run();
?>`


Update ngOptions.js

Check value type to avoid errors

I signed it!

Why is better?

It is better if we use the same function for detecting arrays throughout the codebase, becaue that way it is easier to fix issues or add features.
E.g. right now, isArray() is just a wrapper for Array.isArray(), but it was not always the case (e.g. on older versions supporting IE8) and it might not alaways be the case.
By using the same function, we only have to make the change in one place and we get more consistent behaviour across different parts (e.g. imagine one directive recognizing something as array, while another won't).

Ok, I will change Array.isArray(value) by angular.isArray(value) :)

BTW, you didn't have to close this one. If you pushed a new commit to your branch or amended the previous commit (and force-pushed), GitHub would automatically update the PR.


Implement radial sampling patterns & non-uniform FFT

Modern MRI sampling patterns such as radial sampling need to be implemented.

For this to be implemented, we'd also need to implement some other dataset structures likely, or we need to dive a bit deeper into it.

[ ] Sampling masks will be implemented with  #140
[ ] NUFFT


3 SOL transaction types (_STAKING_DELEGATE, _STAKING_DEACTIVATE, _STAKING_WITHDRAW) not included in Koinly CSV

SOL transactions that only have a fee (i.e. no transfer amount) don't get included in the exported CSV for Koinly. For example, none of these transactions appear in the CSV even though a fee was paid on each of them:
_STAKING_DELEGATE
_STAKING_DEACTIVATE
_STAKING_WITHDRAW
As a result, it is necessary to manually add 'Cost' transactions in Koinly to take these transactions into account.
To clarify: these transactions do appear in the web GUI when viewing the output after analysing the SOL address, but they are not included in the exported CSV file.

In case it helps: an easy way to generate these transactions is to use Exodus wallet to stake your SOL and then unstake it.

Thanks for patience and persistence.  Fixed with https://github.com/hodgerpodger/staketaxcsv/commit/644ad80b3f2fa9073bf9d6a7886e8d7ab0e786d4.  Reopen if continued or related issues.


Exclude from auto-bundles

I tried using RequireJS.Net with a simple mvc.net application using jquery-2.2.0 and just couldn't get it working.

This PR completes the Exclude support for AutoBundles that seems to be missing.

The PR also adds support for selecting CompressionType for the YUICompressor. It defaults to "standard" to not change the existing behaviour, but if you add "compressionType": "none" to an autoBundle the files in that bundle will only be concatenated without being minified.

After putting jquery-2.2.0 in its own bundle and suppressing minification and then excluding jquery from all other bundles, I get the bundles to compile like I want.

I've published your changes on NuGet: RequireJsNet v2.2.4 and RequireJsNet.Compressor v2.2.5
Can you please test that it works on your setup. Thanks for your contribution.

Yes, I confirm the latest upgrades from NuGet works as I’d expect.
There may very well be more issues around this and I added a TODO in the code for future improvements.
I may come around with other PR:s later since this worked out great. :)
/Mattias
Från: Stefan Prodan<EMAIL_ADDRESS>Skickat: den 5 november 2016 01:17
Till: vtfuture/RequireJSDotNet<EMAIL_ADDRESS>Kopia: Mattias Åslund<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Ämne: Re: [vtfuture/RequireJSDotNet] Exclude from auto-bundles (#78)
I've published your changes on NuGet: RequireJsNet v2.2.4 and RequireJsNet.Compressor v2.2.5
Can you please test that it works on your setup. Thanks for your contribution.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/vtfuture/RequireJSDotNet/pull/78#issuecomment-258508642 , or mute the thread https://github.com/notifications/unsubscribe-auth/AFNKNx_abwC8AzJFe7Pgnyv4e-5aHO4eks5q63aHgaJpZM4KpLVC .  https://github.com/notifications/beacon/AFNKN1Rn8vSJ2L4kyp9xdIAPXbYvkiRoks5q63aHgaJpZM4KpLVC.gif


✅ Improve test coverage for core/Heartbeat.py (#121)

Changes

adds: new test for core object Heartbeat

Check off the following

[x] I have tested my changes with the current requirements
[x] My Code follows the pep8 code style.

Could you add the following snipped to the appropriate files which are missing it.
# Copyright Pincer 2021-Present
# Full MIT License can be found in `LICENSE` at the project root.

@Sigmanificient Hey, I'm doing the Hacktoberfest, but seems this PR is not counted. Could you please add the tag  hacktoberfest-accepted to this one?

We hope your pr will get accepted!


adds supports for beerpay! for get donations!

Hi there, I have been using this gem for many years and I found it very usefull, would be nice add this project to https://beerpay.io so we can support you!
thanks for advance!

Tks for your suport! :)
I appreciate it.


Expo error in The minCompileSdk (31)

when i execute the yarn android, im getting this error :

A failure occurred while executing com.android.build.gradle.internal.tasks.CheckAarMetadataWorkAction
The minCompileSdk (31) specified in a
dependency's AAR metadata (META-INF/com/android/build/gradle/aar-metadata.properties)
is greater than this module's compileSdkVersion (android-30).
Dependency: androidx.work:work-runtime:2.7.1

if i change the sdk to 31 i cant build in the turtle service expo.
any help?

Unfortunately this module requires min compile sdk 31. That is not going to change, it is a hard requirement.
If you have some other module that is not compatible with that, you must pursue that with them such that they support it


Maximum (2048) of open files at stdio level appears wrong on W10 x64

I can increase the stdio open file limit to 8192 without receiving an error.

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

ID: 1ac712ff-ab50-25f4-ae4f-04d1b89d48ff
Version Independent ID: 047f00f3-0eec-1177-9759-96584493cbe3
Content: _setmaxstdio
Content Source: docs/c-runtime-library/reference/setmaxstdio.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @corob-msft
Microsoft Alias: corob

@janhec
True enough. Technically, the upper limit is defined by the preprocessor macro _NHANDLE_ defined in corecrt_internal_lowio.h in the Windows SDK sources for the UCRT. IntelliSense tells me its definition is (128 * (1 << 6)), or 8192. I have no idea how old this article is, but the limit hasn't changed since Visual Studio 2015 shipped, so it's been out of date for some time.

Hey Colin,
Thanks for the elaboration!
NHANDLE  appears to be used only in validation, but the important
bits, IOINFO_ARRAYS and IOINFO_L2E cannot be modified without recompiling
the ucrt libs, I guess.
Not that I really need to, but I would be tempted to increase IOINFO_L2E.
Long history of upgrading (I remember my second MB of ram which cost me an
arm and a leg and I had to solder if I remember correctly) tells me that
this limit has become a bit restrictive in comparison with other limits
such as the total number of handles for a proces.
But, no lives at stake here 😉
Sincerely,
Jan
On Wed, May 22, 2019 at 2:43 AM Colin Robertson<EMAIL_ADDRESS>wrote:

@janhec https://github.com/janhec
True enough. Technically, the upper limit is defined by the preprocessor
macro NHANDLE defined in corecrt_internal_lowio.h in the Windows SDK
sources for the UCRT. IntelliSense tells me its definition is (128 * (1 <<
6)), or 8192. I have no idea how old this article is, but the limit hasn't
changed since Visual Studio 2015 shipped, so it's been out of date for some
time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/cpp-docs/issues/1020?email_source=notifications&email_token=ADTBHL5FKRNV6CHZHQIQXTLPWSJLJA5CNFSM4HOJYLDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV5SJFY#issuecomment-494609559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADTBHLZ4CYZ4442JUIEIBYDPWSJLJANCNFSM4HOJYLDA
.

The content is updated, so I'm closing this issue.
Yes, I recall soldering a 512K (K! Not even a megabyte!) memory upgrade directly onto the motherboard of my original Macintosh back in the day. I also remember when I'd build my own modified copy of the CRT from sources, to make a stripped-down version without some of the startup overhead. I suppose that's technically still possible, if you don't need the transcendentals, which don't have source for licensing reasons. There's probably a relevant OS limit on handles that would come into play.
Best,
// Colin


Publisher life cycle view should be consistant

Description:
In the publisher lifecycle view the life cycle state label value is read from "overview_status" attribute of the generic artifact. And the highlighted value in the graph is read from the registry property "registry.lifecycle.<lifecycle_name>.status."
Hence if we change the attribute status from management console and save it, it will show two different values in the lifecycle tab of the publisher

Affected Product Version:
2.1.0 2.2.0

Fixed with PR : https://github.com/wso2/carbon-apimgt/pull/5302


Removing unncessary example

I'm not sure how we deal with PRs here on this mirror as development mainly happens on the gitlab page here (although unfortunately that currently can't take issues/PRs from non-LVC members). @GregoryAshton if this PR looks necessary we should see how we can include it - probably just one of us opening a gitlab MR using the patch from this one.

@vanessagraber does having this example in the repo cause a problem, or does the example just seen to be superfluous?


Include support for therapi runtime javadoc

see https://github.com/dnault/therapi-runtime-javadoc
Javadoc for Java code should be retrievable as well

Example is ProxyMethod Builder in KlumAST, which inherits documenation from the proxied method. Currently, we need the documentation to be present as javadoc and as AnnoDoc annotation, making updates complex. Using an APT and generating json files which in turn are read by AnnoDocUtils methods would solve this neatly.


[BugFix] Hidde public repos owned by private orgs

fix #9555

since i use xorm builder and dont use any sql statement it should be a xorm error ?
@lunny
SearchRepository: Count: mssql: Incorrect syntax near the keyword 'user'.
and
SearchRepository: Count: pq: column "visibility" does not exist

@lunny found the bug user a reserved word like index so I had to add '`'

Codecov Report

Merging #9609 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #9609      +/-   ##
==========================================
- Coverage   42.18%   42.17%   -0.02%
==========================================
  Files         578      578
  Lines       76066    76066
==========================================
- Hits        32087    32079       -8
- Misses      39998    40005       +7
- Partials     3981     3982       +1

Impacted Files
Coverage Δ

models/repo_list.go
74.07% <100%> (ø)
:arrow_up:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

services/pull/temp_repo.go
31.62% <0%> (-2.57%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

routers/repo/view.go
38.59% <0%> (-0.88%)
:arrow_down:

models/webhook.go
70.46% <0%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f54a46...b689efb. Read the comment docs.

@lunny b689efbd945c157f783b42703a784b00a4116a9e did it :)
can you :eyes:  at this bugfix?

The issue title has an extra d in "Hide", maybe you can fix that.

efa0f5461bf44e07e777ddff1a3978d8355cc169 :sweat_smile: if you convert 1 to enum you should use the right value :D
@davidsvantesson I changed the query a bit - hope It is still ok ...?

@davidsvantesson to "Do we know that someone is logged in here, otherwise Limited shouldn't be shown?"
I did a refactor of the code to -> ef3e985676e856fde76c669bb9d11a8a69e7d41c

Hmm... I'm concerned that this might not be the correct place.
https://github.com/go-gitea/gitea/blob/a5a79f702d79b4288d6b214fc9f83ed725edf385/models/repo_list.go#L181-L196
is supposed to limit the repositories to things that can be seen already... So if explore is displaying an inaccessible repo then that's where the problem lies.

OK so /explore/repos points to:
https://github.com/go-gitea/gitea/blob/7e4f490482014fa52d15902b5aa20952461c64f4/routers/home.go#L165-L183
Which means that in the non-logged in case you get:
RenderRepoSearch(ctx, &RepoSearchOptions{
 		PageSize: setting.UI.ExplorePagingNum,
 		OwnerID:  0,
 		Private:  false,
 		TplName:  tplExploreRepos,
 	})

which should map to the false branch above which should be correct and the opts.PublicAll doesn't apply.

OwnerID:  0  && Private:  false
then only this code create the condition:
https://github.com/go-gitea/gitea/blob/master/models/repo_list.go#L186-L195

@zeripath there was no bug with loged out users but if you are loged In you could saw the public repos of private organisations

Then there's a problem with https://github.com/go-gitea/gitea/blob/a5a79f702d79b4288d6b214fc9f83ed725edf385/models/repo_list.go#L181-L186
e.g. with:
https://github.com/go-gitea/gitea/blob/a5a79f702d79b4288d6b214fc9f83ed725edf385/models/repo_list.go#L311-L338

You should only get repositories that you able to see from that condition.

ah I am beginning to re-understand this bloody weird code.

make lg-tm work.

I think that function needs refactoring to always restrict to the accessible repos for the current user.

@zeripath yes this need a refactor ...
schould i backport?


Firefox cant establish secure websocket connection

Hi
Im using https on my site . In firefox this error happens :
Firefox can’t establish a connection to the server at wss://mysite.com:8443/socket.io/?EIO=3&transport=websocket&sid=fAuiKDiXpkhOoSndAAAD
Chrome and IE is ok

i have this problem too


End to end tests are doing something weird with OpenShift logins

@ashetty1
After each make test-e2e on my local machine with Minishift I'm getting this:
github.com/redhat-developer/odo  namespace ✔                                                                                18h17m
▶ ./odo list
User "developer" cannot list deploymentconfigs.apps.openshift.io in project "odo-1527689641"

when I try to go back to using Odo.
Are we doing something funky with oc login? I'm logged into "developer" the entire time, but I think we're doing e2e tests using an administrator account and it's not reverting back to the normal logon after...
Either way, I think that make test-e2e should use whatever account is currently logged in rather than the administrator account, the reason being is that it would be dangerous to do to a production / public cluster if the user runs make test-e2e.

Either way, I think that make test-e2e should use whatever account is currently logged in rather than the administrator account, the reason being is that it would be dangerous to do to a production / public cluster if the user runs make test-e2e.

+1

ping @ashetty1

@cdrage @kadel we specifically do oc login -u developer before running the e2e tests.

After each make test-e2e on my local machine with Minishift I'm getting this:
github.com/redhat-developer/odo  namespace ✔                                                                                18h17m
▶ ./odo list
User "developer" cannot list deploymentconfigs.apps.openshift.io in project "odo-1527689641"
when I try to go back to using Odo.

@cdrage I think that problem that you have there is that e2e tests are creating new projects, and deleting it afterwords.
After last test is run, it deletes the project and leaves it activated in the kubectl config.
You can fix it easily wiht oc project myproject or some other project that you have there.
@ashetty1 could we save current project (oc project -q) before the first test is run and than get back to it after all test are done (oc project)?


Design docs for the yaml file

I'm publishing the first draft, there are a some things that will need more work/review.
Related https://github.com/rtfd/readthedocs-build/issues/29, https://github.com/rtfd/readthedocs-build/issues/30, https://github.com/rtfd/readthedocs-build/issues/3, https://github.com/rtfd/readthedocs.org/issues/3806, https://github.com/rtfd/readthedocs.org/pull/3685, https://github.com/rtfd/readthedocs.org/issues/2904, https://github.com/rtfd/readthedocs.org/issues/2813

Build failure on travis is unrelated

I made some changes based on the feedback, relevant changes are:

The YAML file only have per-version settings (no more globals here)
Use a validation scheme for the spec
Don't cat the YAML in the build process, but show the general settings that were used for the build

Hope I didn't forget anything.

Thanks @stsewd ! This looks great. I think we can start to move conversation on to more specific issues as we begin the planning phase of the work. A lot of this work will start of targeting our 2.5 release, and some of the more specific changes will be on a later milestone.
I think this project will start to be planned in a project for now, as the work will span milestones and is discrete enough to track in one spot. I've created https://github.com/rtfd/readthedocs.org/projects/4 where I'll start prioritizing some of the work that we'll be doing here.

Closing here as i think we've been through this discussion enough here. We'll start by planning thing out a little bit more and setting up some priorities around the work. We have a number of issues to create once the schema is created, so that might be blocking for a little bit.


Add backwards compatibilty for PaginationAmount when upgrading from HC10.4 to HC12

Is your feature request related to a problem?
We are currently in the process of migrating from HC 10.4 to HC 12. Yes, we are late with that :sweat_smile: We now hit a small issue.
A mobile app uses the following query:
query FETCH_SITES($first: PaginationAmount) {
  sites(first: $first) {
  nodes {
    eid
    name
  }
  totalCount
  __typename
  }
}

Variables:
{
  "first": 10
}

But PaginationAmount was removed in HC11, so those queries will throw errors after upgrading. The error message tells me that we should use Int instead of PaginationAmount.
I found instructions regarding backwards compatibility for PaginationAmount in the Stitching section, but those don't apply for what I'm trying to do according to @michaelstaib (clarified via slack).
The solution you'd like
It would be nice if we can migrate from HC10.4 to HC12 without breaking backwards compatibility. I assume that's only possible by keeping PaginationAmount somehow.
Product
Hot Chocolate

Thank you @michaelstaib
For future reference, this can be enabled like this:
services
    .AddGraphQLServer()
    .AddQueryType<QueryType>()
    .SetPagingOptions(new PagingOptions { LegacySupport = true })

@michaelstaib I upgraded now from 12.7.0-preview.41 to v12.7.0 and the LegacySupport field is no longer there. Was it removed by accident?

Or is there now a different way of doing this?


Problem with some processors

Hi, Some people using the gpt4all-ui that relies on your python bindings reported problems with the code hanging at the line that imports the pyllamacpp module.

This happens the same on windows or on linux. It seems that there is something missing in their hardware.

Duplicate of #53


Fix Ninja env for VS 2015 and map toolsets to versions

Addresses: #2516 and #2423

@bobbrow I updated the changelog, would you please take a quick look at it?


Kinematic 3D scene not launched

Hi,
I tryed to run the demo, but it crashes when selecting "New Kinematic 3D Level" option.
After transitioning from "LevelSelect" to "Game/Entry", the game calls a transition, and in the file "StateMachinePlayer.gd" breaks getting the next state:

More information:
OS: Windows 10
Godot version: 4.0.3.stable and 4.1.1
How to reproduce:

Load the project in the editor, and press Run.
Then, press return until "New Kinematic 3D Level" is selected.

Fixed: When the project was imported the "State Machine" field in "res://scenes/game/Game.tscn" scene was not properly loaded.
To fix the error, drag and drop the state machine in "res://AppState.tres" to the "State machine" field in the "res://scenes/game/Game.tscn" inspector.


authenticationToken is not recognized

I have the issue that mý authToken is not being recognized...
Tried several different methodes, checked the code but somewhere the linking between the plugin and the config seems to fail.
Here is the Failure message:
Load homebridge-eWeLink.eWeLink
[2017-12-26 05:58:12] [homebridge-eWeLink.eWeLink] Intialising eWeLink
/usr/local/lib/node_modules/homebridge-ewelink/index.js:42
    this.authenticationToken = config['authenticationToken'];
                                     ^

TypeError: Cannot read property 'authenticationToken' of null
    at new eWeLink (/usr/local/lib/node_modules/homebridge-ewelink/index.js:42:3             8)
    at Server._loadDynamicPlatforms (/usr/local/lib/node_modules/homebridge/lib/             server.js:325:30)
    at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:87:8)
    at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:40:10)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge/bin/homebridge             :17:22)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:149:9)
    at bootstrap_node.js:509:3

and here is my config:
	"platforms": [{

		"platform": "eWeLink",
		"name": "eWeLink",
		"autheticationToken": "authToken is in here!",
		"apiHost": "eu-api.coolkit.cc:8080/api/group?apikey=apikeygoeshere&appid=xyz&nonce=eto149q7&ts=1514263532&version=6",
		"webSocketApi": "eu-api.coolkit.cc:8080/api/group?apikey=apikeygoeshere&appid=xyz&ts=1514263532&version=6"
		},
			]
}

greetings from Germany
Kevin

One thing that I can see in your config is that the URL keys are not set correctly; you only need the hostname component: eu-api.coolkit.cc.
Are you able to show the logs from the start of the launch?
George
Sent from my mobile device

From: coolo19<EMAIL_ADDRESS>Sent: Tuesday, December 26, 2017 6:20 PM
Subject: [gbro115/homebridge-ewelink] authenticationToken is not recognized (#2)
To: gbro115/homebridge-ewelink<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>I have the issue that mý authToken is not being recognized...
Tried several different methodes, checked the code but somewhere the linking between the plugin and the config seems to fail.
Here is the Failure message:
Load homebridge-eWeLink.eWeLink[2017-12-26 05:58:12] [homebridge-eWeLink.eWeLink] Intialising eWeLink/usr/local/lib/node_modules/homebridge-ewelink/index.js:42    this.authenticationToken = config['authenticationToken'];                                     ^TypeError: Cannot read property 'authenticationToken' of null    at new eWeLink (/usr/local/lib/node_modules/homebridge-ewelink/index.js:42:3             8)    at Server._loadDynamicPlatforms (/usr/local/lib/node_modules/homebridge/lib/             server.js:325:30)    at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:87:8)    at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:40:10)    at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge             :17:22)    at Module._compile (module.js:570:32)    at Object.Module._extensions..js (module.js:579:10)    at Module.load (module.js:487:32)    at tryModuleLoad (module.js:446:12)    at Function.Module._load (module.js:438:3)    at Module.runMain (module.js:604:10)    at run (bootstrap_node.js:394:7)    at startup (bootstrap_node.js:149:9)    at bootstrap_node.js:509:3
and here is my config:
    "platforms": [{                                 "platform": "eWeLink",          "name": "eWeLink",              "autheticationToken": "authToken is in here!",          "apiHost": "eu-api.coolkit.cc:8080/api/group?apikey=apikeygoeshere&appid=xyz&nonce=eto149q7&ts=1514263532&version=6",           "webSocketApi": "eu-api.coolkit.cc:8080/api/group?apikey=apikeygoeshere&appid=xyz&ts=1514263532&version=6"              },                      ]}

greetings from Germany
Kevin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/gbro115/homebridge-ewelink/issues/2, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AStTSyoYUKq15H5UozxSdeuaDjWMACpRks5tEIIAgaJpZM4RMgjE.

So I lied... the apiHost does need the port number as well, but nothing after it.
The webSocketHost only needs the hostname.

Okay, i‘ll try it later and give you Feedback If it worked. 😅

okay, just tried it. Thought i found the error because i had a typo in my config.
After correcting it it at least now shows it in my Home app.
But it still fails with the same error as you can see below:
this is from Start:

pi@raspberrypi:~ $ sudo homebridge
*** WARNING *** The program 'node' uses the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see <http://0pointer.de/avahi-compat?s=libdns_sd&e=node>
*** WARNING *** The program 'node' called 'DNSServiceRegister()' which is not supported (or only supported partially) in the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see <http://0pointer.de/avahi-compat?s=libdns_sd&e=node&f=DNSServiceRegister>
[2017-12-26 23:06:24] Loaded plugin: homebridge-ewelink
homebridge API version: 2.2
[2017-12-26 23:06:24] Registering platform 'homebridge-eWeLink.eWeLink'
[2017-12-26 23:06:24] ---
[2017-12-26 23:06:24] Loaded plugin: homebridge-harmonyhub
[2017-12-26 23:06:27] Registering platform 'homebridge-harmonyhub.HarmonyHub'
[2017-12-26 23:06:27] ---
[2017-12-26 23:06:28] Loaded plugin: homebridge-tplink-smarthome
[2017-12-26 23:06:28] Registering platform 'homebridge-tplink-smarthome.TplinkSmarthome'
[2017-12-26 23:06:28] ---
[2017-12-26 23:06:28] Loaded plugin: homebridge-wol
[2017-12-26 23:06:28] Registering accessory 'homebridge-wol.NetworkDevice'
[2017-12-26 23:06:28] ---
[2017-12-26 23:06:28] config.json (/root/.homebridge/config.json) not found.
Load homebridge-eWeLink.eWeLink
[2017-12-26 23:06:28] [homebridge-eWeLink.eWeLink] Intialising eWeLink
/usr/local/lib/node_modules/homebridge-ewelink/index.js:42
    this.authenticationToken = config['authenticationToken'];
                                     ^

TypeError: Cannot read property 'authenticationToken' of null
    at new eWeLink (/usr/local/lib/node_modules/homebridge-ewelink/index.js:42:38)
    at Server._loadDynamicPlatforms (/usr/local/lib/node_modules/homebridge/lib/server.js:325:30)
    at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:87:8)
    at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:40:10)
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:149:9)
    at bootstrap_node.js:509:3
pi@raspberrypi:~ $

maybe youll find the issue
here is my config:
{
	"bridge": {
		"name": "Homebridge",
		"username": "CC:22:3D:E3:CE:30",
		"port": 51826,
		"pin": "031-45-545"
	},

	"accessoires": [{
		"accessory": "NetworkDevice",
		"name": "Büro PC",
		"mac": "xx:xx:xx:xx:xx",
		"ip": "<IP_ADDRESS>",
		"pingInterval": 45,
		"wakeGraceTime": 90,
		"shutdownGraceTime": 15,
		"shutdownCommand": "sfewfeveeegf"
	}],

	"platforms": [{
			"platform": "TplinkSmarthome",
			"name": "TplinkSmarthome"

		},
		{
			"platform": "eWeLink",
			"name": "eWeLink",
			"authenticationToken": "d4300083ce65527b882775750d2794c6a9453ea6",
			"apiHost": "eu-api.coolkit.cc:8080",
			"webSocketApi": "eu-api.coolkit.cc"
		},
		{
			"platform": "HarmonyHub",
			"name": "Harmony Hub Kev"
		}
	]
}

Are you sure that it’s reading your config file correctly?
This makes me suspicious that it isn’t:
[2017-12-26 23:06:28] config.json (/root/.homebridge/config.json) not found.
If it can’t find the file, then all the config values will be null, which would explain the error
Sent from my mobile device

From: coolo19<EMAIL_ADDRESS>Sent: Wednesday, December 27, 2017 11:21:30 AM
To: gbro115/homebridge-ewelink
Cc: George; Comment
Subject: Re: [gbro115/homebridge-ewelink] authenticationToken is not recognized (#2)
maybe youll find the issue
here is my config:
{
"bridge": {
"name": "Homebridge",
"username": "CC:22:3D:E3:CE:30",
"port": 51826,
"pin": "031-45-545"
},
    "accessoires": [{
            "accessory": "NetworkDevice",
            "name": "Büro PC",
            "mac": "xx:xx:xx:xx:xx",
            "ip": "<IP_ADDRESS>",
            "pingInterval": 45,
            "wakeGraceTime": 90,
            "shutdownGraceTime": 15,
            "shutdownCommand": "sfewfeveeegf"
    }],

    "platforms": [{
                    "platform": "TplinkSmarthome",
                    "name": "TplinkSmarthome"

            },
            {
                    "platform": "eWeLink",
                    "name": "eWeLink",
                    "authenticationToken": "d4300083ce65527b882775750d2794c6a9453ea6",
                    "apiHost": "eu-api.coolkit.cc:8080",
                    "webSocketApi": "eu-api.coolkit.cc"
            },
            {
                    "platform": "HarmonyHub",
                    "name": "Harmony Hub Kev"
            }
    ]

}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/gbro115/homebridge-ewelink/issues/2#issuecomment-354019353, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AStTS42izVtJvMfMJFAjVVOnRIYsBeSAks5tEXFqgaJpZM4RMgjE.

Hey,
the issue with config not found is from the wol Plugin because it Needs to run in Root But thats not the Problem, because all other Plugins in the Same config are Running without Root in /Home/pi/.homebridge/config.json...
i think the issue is that the Plugin Cannot recognize the authenticationToken variable in my config:
‘‘‘
TypeError: Cannot read property 'authenticationToken' of null
at new eWeLink ...
‘‘‘

hi, so i think im having the same issue.. just to explain im totally new to all this homebridge stuff and all this programming stuff so please explain as simply as you can... i can pick thing sup pretty quickly and i do know some things. and this is my first device i am trying to set up...
this is the error im getting in CMD:
C:\Users\louis>homebridge
[2018-4-15 18:44:43] Loaded plugin: homebridge-ewelink
homebridge API version: 2.2
[2018-4-15 18:44:43] Registering platform 'homebridge-eWeLink.eWeLink'
[2018-4-15 18:44:43] ---
[2018-4-15 18:44:43] Loaded plugin: homebridge-http
[2018-4-15 18:44:43] Registering accessory 'homebridge-http.Http'
[2018-4-15 18:44:43] ---
[2018-4-15 18:44:43] config.json (C:\Users\louis.homebridge\config.json) not found.
Load homebridge-eWeLink.eWeLink
[2018-4-15 18:44:43] [homebridge-eWeLink.eWeLink] Intialising eWeLink
C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge-ewelink\index.js:42
this.authenticationToken = config['authenticationToken'];
^
TypeError: Cannot read property 'authenticationToken' of null
at new eWeLink (C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge-ewelink\index.js:42:38)
at Server._loadDynamicPlatforms (C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge\lib\server.js:325:30)
at Server.run (C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge\lib\server.js:87:8)
at module.exports (C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge\lib\cli.js:40:10)
at Object. (C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge\bin\homebridge:17:22)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
and this is my config file.    location: C:\Users\louis\AppData\Roaming\npm\node_modules\homebridge
{
"Bridge": {
"name": "HomeBridge",
"username": "CC: 22: 3D: E3: CE: 30",
"Port": 51826,
"Pin": "031-45-154"
} ,
"Description": "HomeBridge HTTP Status Control"
"accessories": [
],
"platforms": [
    {
    "platform" : "eWeLink",
    "name" : "eWeLink",
    "authenticationToken" : "bearer5fe476165d0e22740d2623d4223954e030bae4da",
    "apiHost" : "us-api.coolkit.cc:8080",
    "webSocketApi" : "us-api.coolkit.cc"
    }
]

}
please help... its driving me insane, that i have got this far and im stuck...

@EmmaMay66 I have no experience with running homebridge on a Windows machine, but from what I can see your problem isn't the config.json file but wrong interpretation of JS. In your case "this" can't be found. Speaking very simple - "this" it's a script itself.
I think you should check you npm version and install everything from scratch. Also, check if other plugins work correctly.

Okay. I will try that tonight.
But npm version?
How do I check that? And what should it be?

@EmmaMay66
npm -version

@EmmaMay66  I just keep eWeLink platform and working fine .(remove "platform": "TplinkSmarthome",...etc)

@MrTomAsh what should the npm version be?

Guys just try to copy your 'config.json' to the place home bridge want to.
sudo cp /usr/local/lib/node_modules/homebridge/config1.json /home/pi/.homebridge/config.json

Same Problem here.
TypeError: Cannot read property 'authenticationToken' of null ...
my config.json:
{
"bridge": {
"name": "Homebridge",
"username": ":::::",
"port": 51826,
"pin": "031-45-154"
},
"description": "This is an example configuration file. You can use this as a template for creating your own configur

"accessories": [
],

"platforms": [
    {
    "platform" : "eWeLink",
    "name" : "eWeLink",
    "authenticationToken" : "8f******************************************",
    "apiHost" : "eu-api.coolkit.cc:8080",
    "webSocketApi" : "eu-pconnect3.coolkit.cc"
    }
]

}
Is the problem fixed in the meantime?

2 Minutes later i have found the issue. 💃
Just copied config.json from /homebridge to /.homebridge
Now its working
Command:
cp config.json /root/.homebridge


Add READ_CALL_LOG permission for Android 9

Change-Id: I891d1c6f1be8cab5981c11d43aa4b80755084148

Note: I have only tested this on Android 9 running in the emulator. We will want to test on Android >6, Android >4.2 and Android <= 4.2 to ensure that the inclusion of the READ_CALL_LOG permission doesn't break anything (I don't see why it would but that's why we test).


Integration driver mount retry logic

This patch introduces the ability to configure a retry count and duration for integration driver mount operations. It satisfies a condition encountered by a user with the Ceph driver in concert with K8s.
/cc @cduchesne

Codecov Report

Merging #543 into release/0.6.0 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release/0.6.0    #543   +/-   ##
=============================================
  Coverage           28.4%   28.4%
=============================================
  Files                 34      34
  Lines               2003    2003
=============================================
  Hits                 569     569
  Misses              1373    1373
  Partials              61      61

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1937a13...b4e3050. Read the comment docs.


VKV can't parse KV when reading directly from HttpClient's stream using HttpCompletionOption.ResponseHeadersRead

HttpCompletionOption.ResponseHeadersRead is a special option we can pass to HttpClient which allows to return the response as soon as 200 OK is returned (as opposed to full data), which allows the user to start parsing the data while it's still being buffered.
A repro below:
private static async Task ReproNotWorking() {
    using HttpClient httpClient = new();

    using HttpRequestMessage request = new(HttpMethod.Get, new Uri("https://raw.githubusercontent.com/SteamDatabase/GameTracking-CSGO/master/csgo/resource/csgo_english.txt"));

    using HttpResponseMessage response = await httpClient.SendAsync(request, HttpCompletionOption.ResponseHeadersRead).ConfigureAwait(false);

    KVSerializer? serializer = KVSerializer.Create(KVSerializationFormat.KeyValues1Text);

    KVSerializerOptions options = new() {
        EnableValveNullByteBugBehavior = true,
        HasEscapeSequences = true
    };

    using Stream stream = await response.Content.ReadAsStreamAsync().ConfigureAwait(false);

    serializer.Deserialize(stream, options);
}

Throws:
ValveKeyValue.KeyValueException: Found end of file when another token type was expected.
 ---> System.InvalidOperationException: Attempted to finalize object while in state InObjectBetweenKeyAndValue.

The same repro using HttpCompletionOption.ResponseContentRead instead of HttpCompletionOption.ResponseHeadersRead fixes the issue.
For some reason VKV can't work in this scenario, my blind guess is that it doesn't wait for the end of the stream but rather stops parsing in the middle while more data is being buffered, in result crashing as if the data was cut somewhere in the middle.
The proper logic in this case would include reading stream to the very end, also when being buffered and not fully ready yet.
Could also be something entirely different, I don't know. However, it's nice to note that other deserializing libraries (Newtonsoft.Json tested) work in this scenario just fine.

Is this because we use Peek/Read on StreamReader which is non blocking, or is there another issue at play here?

For reference, I'm working myself with the still-being-buffered stream like the one above in ASF, when downloading huge binary file (23 MB+) and writing it to MemoryStream as it comes.
Dumbed down implementation looks like that:
using Stream stream = await response.Content.ReadAsStreamAsync().ConfigureAwait(false);

MemoryStream ms = new();

await using (ms.ConfigureAwait(false)) {
    ArrayPool<byte> bytePool = ArrayPool<byte>.Shared;

    byte[] buffer = bytePool.Rent(8192);

    try {
        while (stream.CanRead) {
            int read = await stream.ReadAsync(buffer.AsMemory(0, buffer.Length)).ConfigureAwait(false);

            if (read == 0) {
                break;
            }

            await ms.WriteAsync(buffer.AsMemory(0, read)).ConfigureAwait(false);
        }
    } finally {
        bytePool.Return(buffer);
    }

    // Do something with ms ready
}

Nothing really outstanding here, but ReadAsync() for sure waits for more data before returning it, not sure about peek.


JSON API improvements

Following PR adds the following functionality:

Updates the listResponseElement struct to include the fully marshalled Certificate object, Rejected status, request Environment, and request Reason.
For HTTP responses that return JSON data, set the Content-Type header to reflect this.
Add a command line boolean switch (and environment variable) to runserver to support running ssh-cert-authority behind a reverse proxy such as Nginx. When running in reverse-proxy mode, Gorilla's ProxyHeaders middleware is used to inspect X-Forwarded-For and/or X-Real-IP headers to update the remote address on the request object accordingly (along with other information).

All merged in. I also updated the readme to talk about this and the other flags.


Formatted code and added comments

Bruh I forgot to make another branch

damn it wrong account

no MILLION you can't just pick a random change and hit resolved

changes go brrrrrrrrr


Linux Build Instructions Not Current

This is in the docs:
"A build.sh script is provided in Builds/Generated/Linux/Standalone, a symlink of that script is location in Builds/Generated/Linux/VST"
Those files don't exist. Unable to build.
Thanks

Ah, so the stable branch is not the main git branch - I didn't realize this until now
It is still pretty easy find the VST 2 SDK online, just sayin =)

all those files are now on the master branch and should build ctrlr


Module doesn't update when started after octoprint

Hello, thanks for the module !
As said in the title, when my MM (on rasp pi 3) is started before my Octoprint (on a Debian PC, Started only when needed), the module stays N/A everywhere, only a soft restart of MM put everything back as normal.
Am i missing something ?

I have the same issue and so have others...

same here

Please check your configuration to ensure the hostname is correct (try the ip, no port) and that the api key is not in brackets.
Find the config that worked for me below:
            {
                    module: "octomirror-module",
                    position: "middle_center",
                    disabled: false,
                    config: {
                    url: 'http://xxx.xxx.xx.xx',
                    api_key: 'api key no brackets',
                    printerName: 'name for the header',
                    streamUrl: 'http://xxx.xxx.xx.xx/webcam/?action=stream',
                    interactive: false,
                    showDetailsWhenOffline: false
                    }
            },

Please check your configuration to ensure the hostname is correct (try the ip, no port) and that the api key is not in brackets.
Find the config that worked for me below:
            {
                    module: "octomirror-module",
                    position: "middle_center",
                    disabled: false,
                    config: {
                    url: 'http://xxx.xxx.xx.xx',
                    api_key: 'api key no brackets',
                    printerName: 'name for the header',
                    streamUrl: 'http://xxx.xxx.xx.xx/webcam/?action=stream',
                    interactive: false,
                    showDetailsWhenOffline: false
                    }
            },

Does your install show temperatures and so on? I am having the same problem as others where everything shows NA. Only the camera works.

same here must restart mm to show info right


Add option to disable substitutions

👍

ditto that :+1:

This is actually merged into master and pushed manually, but Github is being slow.


libarchive 3.7.4.bcr.2

This adds back a missing part of the patch that was in 3.7.2.
Easier diff for viewing: https://github.com/zaucy/libarchive/compare/b100213b559ad5b1babb74fa7066337b9dfabb7f...cb2a0d5ae1b5fd1ab53f8b717825b1f249a08f31

Hello @dzbarsky, modules you maintain (libarchive) have been updated in this PR. Please review the changes.

Thanks! Wdyt of carrying this as a separate patch file so it's easier to retain between upgrades? And then this PR can also reuse the patch from the previous version?

Yeah I can do that.

Now that BCR supports BUILD overlay I figured it would be cleaner to move in the direction of using that where possible and keeping source patches separate

I'd love to use the overlay feature but wouldn't that mean everyone who uses libarchive 3.7.4.bcr.2 have to use bazel 7.2.0? That's fine with me, but I was hesitant to bring in such a harsh restriction.

Thanks! Wdyt of carrying this as a separate patch file so it's easier to retain between upgrades? And then this PR can also reuse the patch from the previous version?

Yeah I can do that.

Now that BCR supports BUILD overlay I figured it would be cleaner to move in the direction of using that where possible and keeping source patches separate

I'd love to use the overlay feature but wouldn't that mean everyone who uses libarchive 3.7.4.bcr.2 have to use bazel 7.2.0? That's fine with me, but I was hesitant to bring in such a harsh restriction.

That's true, I think we shouldn't do it right now but can do it next time libarchive ships an upgrade so probably in a few months? That's when we'd end up updating the BUILD files anyway


Include takes the relative path of the reader rather than the file's path

Hello, I'm having an issue when trying to parse an included file on a config on a nested folder with this setup:
index.js
const parse = require('@pushcorn/hocon-parser');
const fs = require('fs');

async function main() {
    const file = fs.readFileSync('config/test.conf', 'utf-8');
    console.log(await parse({
        text: file
    }))
};

main();

config/test.conf
{
    include "nest/included.conf"
}

config/nest/included.conf
expected = true

Expected value when running node index.js:
{ "expected": true }

Actual value:
{ }

While testing, I tried setting the path in test.conf to config/nest/included.conf and I got the expected value.
I think the parser is trying to parse the path relative to the "reader" (so in the not working case it would be trying to parse root/nest/included.conf instead of root/config/nest/included.conf). Strangely the CLI seems to be working just fine, as I get the expected result without making any changes (just running npm exec parse-hocon config/test.conf from root).
I didn't see any documentation about relative paths so I'm asking here just in case. I'd appreciate any kind of help if maybe I missed some configuration option or something.
Thanks for the help!

Hello,
In the index file, you are passing the content of the config to the parser, so it'll just use the current directory as the parent directory. If you change the param to { url: "config/test.conf" }, the parser will have the info of the loader path and the relative path will be resolved properly. That's why it works for the CLI.

That did the work, thanks!