Add Globals class so we can avoid passing Context through layers of code

If your class requires a Context instance make it a constructor argument. Then create a static factory method that calls Globals.getContext(). The result can then be passed to the constructor.
This allows testing individual classes using test doubles by directly invoking the constructor and not having to deal with Globals. For integrated tests spanning multiple classes you might have to use Globals.setContext().

Works for me


OKD Implementation

Description
This is work in progress to look into #136
I don't have a test system that allows me going past the download artifacts.
There is a sample-cluster-okd.yml file with the specific settings that are required.
Checklist/ToDo's

[ ] Added documentation?
[ ] Tested?
[x] OKD artifacts download works
[ ] Refactored

@rbo @Alexr03 do you have a system to test?

I have a system to test against, i'll give it a shot now

Initial tests I seem to be getting a failures at:

I believe the condition is incorrect and instead should be: install_okd is not defined

The fcos image is downloaded as a .gz not .xz, the download coreos task is setting the destination file name extension to .gz

I am testing against CentOS 8.
Initial tests I seem to be getting a failures at:

I believe the condition is incorrect and instead should be: install_okd is not defined

The fcos image is downloaded as a .gz not .xz, the download coreos task is setting the destination file name extension to .gz

Cannot seem to SSH/ping into any of the vm's - Running systemctl restart NetworkManager.service will fix this.

Did you set all the variables in the okd sample file?

Yes - This was the cluster.yml I used https://pastebin.com/dLjwGLgJ

"Unzip coreos" should be skipped and "unxz coreos" should be called instead.
Will fix that. What other issues did you spot? Did it download FCOS image and properly load it in libvirt? Did you check the virsh console? Also can you check the dhcp leases?

@Alexr03 pushed a new update that should fix the ".gz"

Turns out the connectivity is fine, it was just taking a while on these stages

It took about 10 minutes to for this to complete. As it is now running the bootstrap.
Not sure if this is normal for how long it normally takes.

@Alexr03 are you on SSD?

@sandrich I am

@Alexr03 not sure how long it took on my end. Let's see if you can ssh to bootstrap and if the cluster is forming.

Can confirm I can ssh into bootstrap and its forming the cluster.

@Alexr03 ideally you destroy and recreate from scratch with my updated code once the cluster is up and running. Just so I can test it end-to-end

I let the bootstrap run through anyway and when it reached registry it failed due to SSL.

I will destroy and recreate the cluster now to test e2e.

Same error after destroying and recreating unfortunately.

Same error after destroying and recreating unfortunately.

Dns error or extraction error?

The ssl error for accessing the cluster api. - https://github.com/RedHat-EMEA-SSA-Team/hetzner-ocp4/pull/137#issuecomment-716733858
Maybe linked to #117 ?

Not sure this is related to OKD @rbo ?

Managed to get it all up and running after trial and error alot of times. Not sure what was causing it, will do a little bit more investigating into it.
I let the cluster settle and it seems that everything is working as expected apart from the community-operators pod in openshift-marketplace. Seems to be getting CrashLoopBackOff error. I think I remember seeing this when I installed ocp4. I don't think its down to okd.
Checking out the logs, I don't see any obvious reasons why its crashing 🙃

@rbo PR is ready for review

24 hours later --
Restarted the cluster and it seems the apiserver will not come back up, presumably due to a invalid certificate on the api.cluster.domain.tld
I am getting EOF errors when attempting to login,
[root@static hetzner-ocp4]# unset KUBECONFIG
[root@static hetzner-ocp4]# oc login -u Alex
Server [https://localhost:8443]: https://x.y.z.a:6443
The server uses a certificate signed by an unknown authority.
You can bypass the certificate check, but any data you send to the server could be intercepted by others.
Use insecure connections? (y/n): y

Unable to connect to the server: EOF

I am able to see the cluster is up via looking at the <cluster ip>:6443/version

just for information: it will take some time to review this.

Everything is merged into okd4 branch for further tests.


Terminal cursor stays in the same line instead of moving to the bottom

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.60.1
OS Version: Windows 10 Enterprise

Steps to Reproduce:

Quite often when stop your server (Ctrl+C), then realize that cursor stays in the same command line instead of moving to the bottom
see screenshot attached below

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to our code. As such, I will go ahead and close it. Please try to reproduce in Windows Terminal and report to microsoft/terminal with repro steps.
If you cannot reproduce in Windows Terminal, chances are it's fixed in a later version of Windows. You can read more about how we consume conpty here. Thanks for understanding.
/label upstream terminal-conpty windows


inform6: 6.41-r5 -> 6.41-r6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/inform6/versions.
meta.description for inform6 is: Interactive fiction compiler and libraries
meta.homepage for inform6 is: https://gitlab.com/DavidGriffith/inform6unix
meta.changelog for inform6 is: https://gitlab.com/DavidGriffith/inform6unix/-/raw/6.41-r6/NEWS
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.41-r6 with grep in /nix/store/gv4k4j95ixqyp5nq7crrj4pwp8hads14-inform6-6.41-r6
found 6.41-r6 in filename of file in /nix/store/gv4k4j95ixqyp5nq7crrj4pwp8hads14-inform6-6.41-r6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
inform6

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gv4k4j95ixqyp5nq7crrj4pwp8hads14-inform6-6.41-r6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A inform6 https://github.com/r-ryantm/nixpkgs/archive/7e0f542df82a92384f7d0d7cb63630efab53a95d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gv4k4j95ixqyp5nq7crrj4pwp8hads14-inform6-6.41-r6
ls -la /nix/store/gv4k4j95ixqyp5nq7crrj4pwp8hads14-inform6-6.41-r6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

inform6

Maintainer pings
cc @ddelabru for testing.

Result of nixpkgs-review pr 244213 run on aarch64-darwin 1

1 package built:

inform6


Update to Pyodide 0.20.0

This hasn't been done yet for reasons described in this pyodide-worker-runner issue

Depends on a pending PR in pyodide-worker-runner, more information has been requested there.

The current state of the latest Pyodide-version isn't exactly ideal as they are still figuring out a couple of important bundling issues. When deemed useful, pyodide-worker-runner will be updated, at which point Papyros can be upgraded too.


Revert "spawn_thread with atomic operation"

This reverts commit 05cc3df86672cbb16f17e8909c8b471fac618d4a.

Checking to see if autest passes by reverting this commit.

latent issues here ? or just autest script issue ?

Yes, it's quite mysterious why this would break AUTest.

is the autest from scratch with no storage data etc? and is stats data cleared before the run?

Yongming Zhao 赵永明

在 2017年5月26日，下午9:39，Alan M. Carroll<EMAIL_ADDRESS>写道：
Yes, it's quite mysterious why this would break AUTest.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/apache/trafficserver/pull/1984#issuecomment-304284960, or mute the thread https://github.com/notifications/unsubscribe-auth/AA1DXFlFJWI6pi_xvpp9PDeSYE2qIY_Bks5r9tYngaJpZM4NnJd8.

Hey, Emperor Ming! Yes, AUTest creates a sandbox directory for each run which should be created on the fly, so no history effects. My observations are that the proxy port isn't get set but why this change should affect that is unclear to me.

It is unclear to me, but it seems like autest started having issues after this commit.  I decided to do a revert and test it in our CI system to see if it would resolve the issue.
Looks like CI passed.  I am going to rerun CI a few times and see if it passes each time.

[approve ci]

[approve ci]
One more time...

[approve ci]
au test succeeded 2x and failed 1x - running again...

[approve ci]

--- a/lib/records/RecRawStats.cc
+++ b/lib/records/RecRawStats.cc
@@ -47,7 +47,14 @@ raw_stat_get_total(RecRawStatBlock *rsb, int id, RecRawStat *total)
     total->count += tlp->count;
   }

-  for (i = 0; i < eventProcessor.n_dthreads; i++) {
+  // The DEDICATED EThread can be created during ATS running.
+  // In the eventProcessor.spawn_thread(), the n_dthreads is incremented first
+  // then create the EThread object and put it into the array all_dthreads[]
+  // Therefore, the all_dthreads[eventProcessor.n_dthreads] maybe nullptr.
+  int n = eventProcessor.n_dthreads;
+  if (eventProcessor.all_dthreads[n] == nullptr)
+    n--;
+  for (i = 0; i < n; i++) {
     tlp = ((RecRawStat *)((char *)(eventProcessor.all_dthreads[i]) + rsb->ethr_stat_offset)) + id;
     total->sum += tlp->sum;
     total->count += tlp->count;

PR #1989 is created to resolve this issue.

Not needed anymore


Test against PHP 7.1

Coverage remained the same at 62.567% when pulling b46ff8f4551c0e50d35d151b0953177c16edf610 on acorncom:patch-1 into c3579440a3ca47ca45bfb0ed854bc0ff9b086bf5 on rcrowe:master.

Coverage remained the same at 62.567% when pulling b46ff8f4551c0e50d35d151b0953177c16edf610 on acorncom:patch-1 into c3579440a3ca47ca45bfb0ed854bc0ff9b086bf5 on rcrowe:master.

Coverage remained the same at 62.567% when pulling b46ff8f4551c0e50d35d151b0953177c16edf610 on acorncom:patch-1 into c3579440a3ca47ca45bfb0ed854bc0ff9b086bf5 on rcrowe:master.

Coverage remained the same at 62.567% when pulling b46ff8f4551c0e50d35d151b0953177c16edf610 on acorncom:patch-1 into c3579440a3ca47ca45bfb0ed854bc0ff9b086bf5 on rcrowe:master.

I missed the line that allows PHP 7.0 test failures, will close this for now


LDC compatibility

LDC for the moment is incompatible with a GCC C/C++ backend. The cause of the issue is error/exception handling.
I will need to write glue code to accept LDC's compiler generated exception-code and make it interact with GCC's (not GDC!!) exception code.

This is LDC's exception handling implementation and you'll notice that it is incompatible with our GDC-based exception handling. For refence, this is GDC's execption handling code.


Broken with Bazel@HEAD on Bazel CI due to --incompatible_remove_native_maven_jar

Bazel@HEAD (and the upcoming Bazel 2.0) will flip --incompatible_remove_native_maven_jar to true by default.
Building //... breaks with that flag enabled.
Repro:

build Bazel at https://github.com/bazelbuild/bazel/commit/8f075d238fa099950b470da2541548d932b2b535
use that to build //... at https://github.com/bazelbuild/bazel-integration-testing/commit/cd4f16d50898eebe0eca0c09496fe15e9a642f71
the build fails, but if you add --noincompatible_remove_native_maven_jar then it works

Thanks!
I’ll handle it soon
On Tue, 26 Nov 2019 at 14:37 László Csomor<EMAIL_ADDRESS>wrote:

Bazel@HEAD (and the upcoming Bazel 2.0) will flip
--incompatible_remove_native_maven_jar to true by default.
Building //... breaks with that flag enabled.
Repro:

build Bazel at bazelbuild/bazel@8f075d2
https://github.com/bazelbuild/bazel/commit/8f075d238fa099950b470da2541548d932b2b535
use that to build //... at cd4f16d
https://github.com/bazelbuild/bazel-integration-testing/commit/cd4f16d50898eebe0eca0c09496fe15e9a642f71
the build fails, but if you add
--noincompatible_remove_native_maven_jar then it works

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/issues/158?email_source=notifications&email_token=AAKQQF32VGW4FAOJNHH4PM3QVUKARA5CNFSM4JRXLRNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H4DJDVA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKQQF3RVVOOPFYMHUJFA4TQVUKARANCNFSM4JRXLRNA
.

--
Ittai Zeidman
Cell:<PHONE_NUMBER>
40 Hanamal street, Tel Aviv, Israel
http://www.wix.com

Closed by the amazing @jin #159


add class diagram - delete model_summary.txt - change cnn_cat_dog -> Alexnet.h5

Could you please make svg a separate file (e.g. in docs folder)?


Dart plugin available ?

Is it yet possible to generate Dart code from this generator ?
I try this: https://github.com/micimize/graphql-to-dart
But I get an error: TypeError: Must provide Source. Received: {}

Hi @fvisticot !
graphql-to-dart package is using the old API of the codegen. You can try to use it with an old version (0.9.2), or contact @micimize in order to update it to the latest API.

@fvisticot I don't believe there are any robust dart generators yet. You can try using mine with the older generator - I should hopefully have time to update it soon

<EMAIL_ADDRESS>should now work with 0.18.0

Hello there, please check out the generator  I wrote for Flutter to generate Freezed models from your GraphQL types #8171.
Let me know if you need any features. I am actively working on it cause I need it for some projects and I hate to manually write Freezed models when my backend keeps changing frequently.


Cannot define custom stdout/stderr output

I'm having an issue getting a custom output to work.
My package.json has the following start script:
"start": "npm run server & npm run batch-deploy",

"server": "node server/server.js >> /var/log/dokku/studio-sensors/node.log 2>> /var/log/dokku/studio-sensors/node.err",
"batch-deploy": "node server/batch-deploy.js >> /var/log/dokku/studio-sensors/batch-deploy.log 2>> /var/log/dokku/studio-sensors/batch-deploy.err"

I would have thought that since /var/log/dokku/studio-sensors is being mounted as a volume, it would be accessible inside the container, however, I'm getting the following error when running:
sh: 1:
cannot create /var/log/dokku/studio-sensors/node.log: Directory nonexistent

Am I doing something wrong, or is this not possible?
Alternatively, is there another way to get the 2 processes to output to independent files? Less concerned with stderr and stdout being separate, as much as having the server task and batch-deploy task have separate output.

I believe the aforementioned directory is actually mounted as /var/log/app inside the container.

See here: https://github.com/sehrope/dokku-logging-supervisord/blob/master/docker-args

Cool thanks. Let me give that a try! I forgot that it might be mounted with a different name.

Awesome. Yup that was totally it! Thanks for the quick reply!


Help! Part 5 - Section 2

Hi Plailect,
I couldn't access this link:  http://dukesrg.github.io/2xrsa.html?arm11.bin
the browser will just freeze half way, is there any work around,  so that I don't need to visit this link?
Thanks

Did you copy arm11.bin to your sd card?

It's in the safea9lh file

yes yes, I did all the steps written on the guide

anyway, I have a cubic ninja copy, if this helps to do other work around...

try it a few more times it can take a bit.
You could also reset the browsers data ever time you try it to increase the chances.

somehow got it work, copy and replacing the same file and I got the glitched screen, pressed select to install and everything works like a charm =)
Thanks @nickthegamer5


initialize/new methods don't show up in docs anymore

class CC
  # asdf
  def initialize(a : Int32)
  end
end

This method used to be shown in generated API documentation, but not anymore.

The first bad commit could be any of:
b01fc5396d17b4a781f2c01db16c45977cecaa33
a49aef7f45268be005a045f4aec2233114e68686

@BlaXpirit Thanks! I didn't notice this at all...


refactor(clippy): apply semicolon_if_nothing_returned clippy lint

Description
Apply semicolon_if_nothing_returned clippy lint from the pedantic group.
Motivation and Context
Some of the checks from pedantic group are very useful to apply into the project. I will select the useful ones, and apply each to own MR, for maintainer to easy see the changes and easier decided which he want to apply and which not.
I think it's useful to apply them every once in a while, but not to use them by default.
I use the refactor(clippy) format for my commits, which is omitted from the changelog report.
I also apply those changes to get more familiar with the project and later I will take some of the open issues.
How Has This Been Tested?
I ran the specific lint checks and solved the problem until the linter no longer told me any issue.
Command:
cargo clippy --all-targets -- -D warnings -W clippy::semicolon_if_nothing_returned

The cargo build and test passed successfully. Standard usage of linter and formatter also passed.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[x] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 39.94%. Comparing base (a82dac3) to head (0c9d470).

Files with missing lines
Patch %
Lines

git-cliff/src/lib.rs
0.00%
4 Missing :warning:

git-cliff-core/src/commit.rs
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #851   +/-   ##
=======================================
  Coverage   39.94%   39.94%
=======================================
  Files          20       20
  Lines        1660     1660
=======================================
  Hits          663      663
  Misses        997      997

Flag
Coverage Δ

unit-tests
39.94% <37.50%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@nejcgalof would you be also interested in submitting a PR for enabling the pedantic clippy checks in CI? I'm guessing it should be either a warn annotation in lib.rs or some command line argument addition to cargo clippy command.

@orhun Of course. I will do it :)


Exception during git bigstore pull

...
  File "/usr/local/lib/python2.7/dist-packages/bigstore/bigstore.py", line 282, in pull
    g().notes("--ref=bigstore", "add", "-m", "bigstore")
  File "/usr/local/lib/python2.7/dist-packages/git/cmd.py", line 450, in <lambda>
    return lambda *args, **kwargs: self._call_process(name, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/git/cmd.py", line 902, in _call_process
    return self.execute(make_call(), **_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/git/cmd.py", line 695, in execute
    raise GitCommandError(command, status, stderr_value)
git.exc.GitCommandError: 'git notes --ref=bigstore add -m bigstore' returned with exit code 1
stderr: 'error: Cannot add notes. Found existing notes for object fc559df2dbc6a822d30b4a4ab498e3681ffe0155. Use '-f' to overwrite existing notes'

Did you ever figure this out?
I'm getting the same error. When I try removing notes for named object, git tells me no note exists (and there shouldn't be any yet).

I believe this was fixed in #36 (it removes the offending line and merges it with an equivalent line that catches the exception).
Are you running the latest version?  If so, post your stack trace.

Thanks so much. I'm not exactly sure how to check the version explicitly, I couldn't find info on the wiki?
I installed bigstore through pip yesterday so I'm not sure if the latest version is updated there. I can post the stack trace if so though.

I'm checking right now, let me see what's currently up there and if @jansel's fix made it in.

Thanks so much for your help. Pip may not install or upgrade to the latest bigstore version, metadata.py seems to indicate 1.0.0

Looks like PyPi changed their package updating API. Hold tight while I tighten the screws…

Done!


Implement a Dev mode

To help the developers to work on the project but without having to handle the hassle of setting up the authentication using OIDC, we need to implement a development mode to the gitops server.
Implementation Note:

It should be a middleware similar to https://github.com/weaveworks/weave-gitops/blob/v2/pkg/server/handler.go#L45 that will inject the Principal but using WEAVE_GITOPS_AUTH_DEV_USER and WEAVE_GITOPS_AUTH_DEV_TEAMS environment variables as values.

With the super admin user this should not be necessary

👍  I already have this as part of https://github.com/weaveworks/weave-gitops/issues/1454

@luizbafilho are we okay to close this?


Configure port number listened by the websocket in setting file

First of all, many thanks to you for developing this super awesome plug-in. I have been searching for how to co-work on TiddlyWiki for quite some time and this plug-in totally solves my problem. Not to mention that the auto-saving functionality comes with it is also very handy.
Now I am trying to host a TiddlyWiki server with this plug-in in a docker container. I have to determine which ports to expose from the container for the server to work correctly. However I find that although I can configure the serving port in the settings.json, but I cannot find a place to configure the listening port. Is it possible to configure it or currently is it safe to assume that it will always be the serving port + 1?

Thanks a lot for your reply. I think in my use case it is good enough for me to know that the port number will be next open port greater than the server port. I opened this issue simply because I was curious if there is a way to configure it but not really wanted to request for such a feature if it is not possible. So please feel free to close issue if you would like to.

This should be implemented now. In the settings.json file if you set 'autoincrementPort' to false under 'ws-server' than the web socket server port will be the one above the wiki port and the process will fail if the port is in use.
By default (if the entry is missing in the settings file) than the server will use the next open port. The example settings file has it set to false to show where the setting is.
This does mean that while that is set you will have to manually list the ports in the configuration files for any child wikis you start that have the multi-user plugin active.

@a3785lex May I ask you for your docker setup with TiddlyWiki and Bob Plugin? I'm struggling with setting up my own.
BTW Thank you for this wonderful plugin Jed and sorry for hijacking an old thread :grimacing:

I don't use docker. In the future.make a new issue, don't make unrelated commments on old issues.


Feature: Create API for Todo List Application

Description:
The Todo List application currently operates as a console-based application, limiting its use to local environments. This issue proposes the creation of an API to extend the usability of the application beyond the terminal. The API will allow users to interact with the Todo List programmatically, enabling integrations, remote access, and third-party applications.
Proposed Solution:
Design and implement a simple API for the Todo List application. The API should provide endpoints for common operations such as adding, deleting, completing tasks, and retrieving task lists. Consider standard HTTP methods (GET, POST, DELETE) for CRUD operations. Define a clear and user-friendly API documentation to guide developers in using the endpoints.
Additional Notes:

Use a lightweight web framework suitable for the chosen programming language (e.g., Flask).
Include authentication mechanisms to secure API endpoints.
Consider implementing versioning for future updates.
Discuss potential use cases for the API, such as integration with task management tools, mobile apps, or web applications.

Hello, I'd like to try work on this. What kind of authentication service would you like me to use? Perhaps I could add a user account system to the main app, and implement that in the API aswell?

I was thinking of a Flask API, and I see that you already worked on it. I don't ready know a lot about APIs, but I trust you on that. As you said, an account system is a good idea. For me, the main idea of the API was about being able to open the app in a more user-friendly way, on a web page or a new window.


Fix Typescript Lint error basic.

I fix typescript lint error.
ISSUE=#467

I request it, but we have more lint error in server folder!

@yjaeseok, could you update commit description? You seem to remove that ignore cases?
(Patch looks good to me.)

@romandev Okay, I updated commit description.
Next, we should discuss about ignore rules.


iframe style injection

Description
Users want to be able to inject styles into iframes for some use-cases.
We should leverage the iframe API to support this.
Proposals
Option 1
Consider leveraging the initialContent API from auto-frame-component:
<Puck iframe={{
  initialContent="<!DOCTYPE html>

"
}} {/* ... */} />

Option 2
Use a stripped back version of initialContent API from auto-frame-component:
<Puck iframe={{
  head=""
}} {/* ... */} />

Option 3
Add a dedicated parameters
<Puck iframe={{
  style="body { background: hotpink; }"
  styleUrl="https://www.example.com/styles.css"
}} {/* ... */} />

My first pick would be Opt 1 for maximum flexibility, though I'd be okay with Opt 2 as well.

Ideally for my use-case, this would be dynamic, so updating the props would work (this is isn't clear given the name 'initialContent').

After considering opening a PR for this, I came across some potential challenges based on my use case, particularly for working with tailwind.
For anyone who want to offer users the ability to set tailwind classes on components and compile JIT style at runtime - mirroring the host styles could introduce classname conflicts. The typical way of using tailwind & JIT compilation w/ page builders is to load the tailwind play CDN into the iframe:
But if the host app is also built with tailwind, then the host's globals.css file would clash with the css that tailwind play's script generates. There needs to be some style isolation.
I looked at the code where the styles are being mirrored, and after testing, realized that puck.css and global.css all get injected together in a style tag in the host's layout file.
Then I tried adding to @measured/puck's build script by additionally generating a module that exports a raw string of puck.css to do something like this:
import puckStylesRaw from "../dist/raw-styles"

const AutoFrameComponent = React.forwardRef<HTMLIFrameElement, AutoFrameProps>(
  function ({ mirrorHostStyles: boolean, ...props }: AutoFrameProps, ref) {

    return (
      <Frame {...props} ref={ref}>
        <CopyHostStyles puckStyles={puckStylesRaw} mirrorHostStyles={mirrorHostStyles}... >

But it started feeling clunky and I realized I may be overcomplicating it. 😅
Ultimately, my goal was to keep the puck.css styles in the frame since they're needed, while making the rest of the host styles optional to include.

@nicolas-angelo @4leite do you need style isolation in the Puck editor, or just in the final rendered output?

@chrisvxd both. Although I know it's easier to manage styles for rendered output. The problem is the mirrored styles showing up in editor.
I plan on loading the tailwind play CDN in the editor and letting users enter dynamic classes, so I don't want the tailwind classes (or any classes from my host global stylesheet) ending up in the editor.
Im less concerned about isolating the puck.css classes because I don't really expect that to clash.
But preferably the editor can be used as a clean slate for styles.

I have the above working. (Tailwind play CDN and users entering dynamic classes).
I actually don't need style isolation as I deal with that using tailwinds class prefix option.

@4leite that dawned on me the other day but I still felt uneasy having the host styles mirrored in by default, even styles that weren't specifically tailwind generated.
Awesome that you got it working though! Feel free to share you're approach if you don't mind it being public 😅

You can see the gist of it there, although I haven't merged the class isolation and some bug fixes.
https://github.com/Tohuhono/Oberon/tree/main/packages/ui/src/theme

@chrisvxd  This is not a priority feature for me personally (I have solved it for my use case), I was more chiming in to document how I solved it for what seems like a similar use case.

@chrisvxd yea I think I'm going to work off @4leite solution. But as far as general style injection feedback, I'd be happy with the following

renaming initialContent to 'content' or something that implies the value being dynamic and not initial
option to skip injecting host styles in iframe (except for the necessary puck styles)

Option 4
Include CSS in the Puck data? Something like:
{
  "content": [],
  "root": {
    "props": {
      "styles": "body { background: hotpink; }"
    }
  },
  "zones": {}
}

Reasoning: We are planning to have a CSS editor, which would update the styles in the data object. Currently we grab the styles prop and apply it inside the iframe body by overriding root render and placing a style element before children.
const config: Config = {
  root: {
    render: ({ children, styles }) => {
      return (

          {children}

      );
    },
  },

While this works, it would be better, if the styles got updated in the iframe head. In case of no iframe the styles could just go in the main window head I suppose...

Great workaround @jarrrgh!

Does this prop will be included in <Puck.Preview /> component?

I'm now also facing an issue due to Tailwind styles (and other styling) ending up inside the iframe. It would be helpful to have a way to control which styles are included in the iframe, in addition to the necessary Puck CSS, of course.

I've implemented a PR based on option 1.  While it isn't dynamic I thought it could be paired with the new onAction to help with that. e.g.
<Puck
  iframe={{
    enabled: true,
    initialContent: '<!DOCTYPE html>

'
  }}
 onAction={(action, appState, prevAppState) => {
   if (action.type === 'insert') {
     document.getElementById(#styles).innerHTML( SYNC STYLES HERE)
   }
 }}

@4leite and @nicolas-angelo I'd be curious to know if this helps?

Ended up taking a different approach to this as part of #543 via the new iframe override API:
iframe override API
Receives the iframe document as a prop. This enables you to inject your styles and manipulate the iframe document however you need.
    <Puck overrides={{
      iframe: ({ children, document }) => {
        useEffect(() => {
          if (document) {
            // inject styles
          }
        }, [document]);

        return <>{children}</>;
      },
    }} />

Because this is an override, we can create plugins for common use-cases, like the new emotion-cache plugin.
Going to close this out as I think this API should cover most use-cases, but @ me if it's missing something.


Bug: Resulting multiline replacement string duplicated

Input file:
/some/path/config
/some/other/config

/some/path/config
/some/other/config

GrugFar commands:
Search:
/some/path/config
/some/other/config
Replace:
/some/path/config
/some/different/config
Files Filter: # default, not changed
Flags:
--fixed-strings --multiline # or -U

Action:
<leader>r

Current result:
/some/path/config
/some/different/config

/some/path/config
/some/different/config

/some/path/config
/some/different/config

Expected result:
/some/path/config
/some/different/config

/some/path/config
/some/different/config

History entry:
Engine: ripgrep
Search: /some/path/config
| /some/other/config
Replace: /some/path/config
| /some/different/config
Files Filter:
Flags: --fixed-strings --multiline
Paths:

Comment:
For some reason when replacing multiple lines, extra lines from "Replace:" appear at the end of the file.

hi @mjrogozinski, thanks for reporting! Just tried it though and I cannot reproduce it :confused:
I am no ripgrep 14.1.0. What version are you on?

I have checked it with 14.1.0 on a mac and indeed you are right. The problem does not occur.
As for the version used earlier, I will have to check during the weekend. It was on my home Linux PC.
Still, let's close this issue for now.

Indeed wrong ripgrep version was to blame. Updating 13.0.0 to the newest fixed the issue.
The plugin works like a charm. Thank you!


Improve View editor

Overview
Currently, View (SQL) editor has been bootstrapped and can be run in the application or as an isolated component via npm run components as Editors/View. We'd like to have a lot of nice features here.
Components
The only components affected by this work:

Editors/View
Parts/* (if needed)

Actions

[x] start working on the component in isolation npm run components
[ ] add table's file path to the Fields (for a table node show at the right in grey)
[ ] add field's type to the Fields (for a field node show at the right capitalized)
[ ] add a Fields search/filter (filtering by table name or field name)
[ ] add query validation (use lib like this - https://github.com/JavaScriptor/js-sql-parser) (and add a compact error feedback)
[ ] consider adding "Format" query button using SQL parser from above
[ ] try inserting a field name to Query on a double click on a field name in Fields
[ ] fix geometry/scrolling/etc (available and fixed window height: theme.spacing(42))
[ ] consider adding a helpline (some context help to the user)

[ ] add table's file path to the Fields (for a table node show at the right in grey)
[ ] add field's type to the Fields (for a field node show at the right capitalized)

I can't see a simple way to add these information in the same line as the tree node label. It's easy to change the label text, but if we want to have more control in how we display the tree nodes we will need to change:
https://github.com/mui/material-ui/blob/master/packages/mui-lab/src/TreeItem/TreeItemContent.js
What do you think? We create our own version of the Tree View (or just the component TreeItem)?

What do you think about having syntax highlighter also? https://www.npmjs.com/package/react-syntax-highlighter

@aivuk
We already have a few of trees - https://github.com/frictionlessdata/application/tree/main/src/components/Parts/Trees. I think we can tweak the BaseTree the similar way we did for FileTree. But also maybe we can use ContentComponent prop from https://mui.com/material-ui/api/tree-item/?
@shashigharti What would you suggest?

Syntax highlighter is a good idea we can start from a simple one and then migrate to Ace's highlighter (or whatever editor we use for test editing)

@aivuk We already have a few of trees - https://github.com/frictionlessdata/application/tree/main/src/components/Parts/Trees. I think we can tweak the BaseTree the similar way we did for FileTree. But also maybe we can use ContentComponent prop from https://mui.com/material-ui/api/tree-item/?
@shashigharti What would you suggest?

thanks! ContentComponent will do the job :+1:

@aivuk We already have a few of trees - https://github.com/frictionlessdata/application/tree/main/src/components/Parts/Trees. I think we can tweak the BaseTree the similar way we did for FileTree. But also maybe we can use ContentComponent prop from https://mui.com/material-ui/api/tree-item/?
@shashigharti What would you suggest?

I agree with @roll there might be a way to tweak BaseTree for the current usecase.

add a Fields search/filter (filtering by table name or field name)
try inserting a field name to Query on a double click on a field name in Fields

for now in my draft branch I'm working in validating the fields and table names before sending the query to the api. I'm also found out how to use the ACE editor API to insert some text in the cursor. Do we have a standard way to do the communication between two different components? In this case I would need to send a message from the FieldsTree to the Query.

fix geometry/scrolling/etc (available and fixed window height: theme.spacing(42))

what is this problem exactly, sorry, I couldn't get it.

@roll regarding:

fix geometry/scrolling/etc (available and fixed window height: theme.spacing(42))

I could only fix having a fixed height where the sql editor and fields tree is placed.

@roll

add a Fields search/filter (filtering by table name or field name)

I did the search with a the search input above the fields tree. The search logic is done, and I started to implement how to display the results. Would you prefer to display the results in a separate component, or do I change the FieldsTree component to highlight the fields and table names?


Add Browser Analytics

We'll want to add Google Analytics or something so that we can see traffic and browsers and such. This will most likely largely be done in pypi-theme.

Surely we want this to be active on launch?

Yea we want this active on launch, but it's trivial to do so it shouldn't be hard, it'll just be dropping in a bit of JS into the template. The only hard part is our CSP policy.

I asked for some clarification in IRC in #pypa-dev from Donald just now, and learned:
We already have metrics on legacy PyPI, so it's important to have analytics active in Warehouse at launch, so we don't miss data during the switch.
To abide by the content security policy, we need to figure out how to do GA without chunking some inline JS; our CSP policy instructs browsers to reject inline JavaScript in the HTML, only [allowing] linked scripts. In the worst case, we could relax the restrictions, but Donald would rather not do that.


fix name generation for NV extensions

Hey pqnet,
thanks for the patch. Could you please sign the CLA above so that I can merge it?

Thank you!


Log account and transaction

@evgenykuzyakov said:
I want to make a request for near-shell. I think we should always log the transaction hash whenever shell makes any transaction. It will help us go to explorer and debug it in case something goes wrong
Other thoughts:
We should also start using account ID + transaction hash as a transaction identifier as otherwise it is tricky to get the result from the node

Agree, we should link to explorer for transaction issued and for transactions that create/change accounts we should link to them as well.
It helps with checking status on tx, debugging, with promoting overall explorers as a tool that people should use.
Explorer should be part of configuration in .neardev/config.json to identify which explorer to use with something like this:
"explorer": {
   "mainnet": "https://explorer.nearprotocol.com",
   "betanet": "https://explorer.betanet.nearprotocol.com",
    ...
   "account_url": "/accounts/{account_id}",
   "tx_url": "/transactions/{tx_hash}",
}

duplicate https://github.com/near/near-shell/issues/268


Authless parsed url host support

0.5.11 changes slightly the objects returned by url.parse.
This makes request not get confused by the new style.

Oh, apparently also other stuff that was sitting on my master branch waiting to be merged.


Update hints.yml

Added php snippets

Thanks for the work!
The snippets will be included in the next version, cheers :beer:


"ng update" can install @next packages without the --next flag (Get compileFactoryFromMetadata is not a function after upgrading to angular 8)

Hi i can't run my project after updating it from angular 7 to angular 8 by running ng update @angular/cli @angular/core
🔥 Exception or Error

chunk {main} main.js, main.js.map (main) 2.02 kB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 149 kB [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 6.15 kB [entry] [rendered]
chunk {scripts} scripts.js, scripts.js.map (scripts) 147 kB [entry] [rendered]
chunk {styles} styles.js, styles.js.map (styles) 1.63 MB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 339 kB [initial] [rendered]
Date: 2019-10-04T16:50:24.470Z - Hash: 65f8616cbfb2cdc52e74 - Time: 6122ms

ERROR in compiler_1.compileFactoryFromMetadata is not a function
** Angular Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **
i ｢wdm｣: Failed to compile.

🌍  Your Environment
Angular Version:

Angular CLI: 8.3.8
Node: 10.16.3
OS: win32 x64
Angular: 8.2.9
... animations, common, compiler, core, forms, language-service
... platform-browser, platform-browser-dynamic, router
... service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.803.8
@angular-devkit/build-angular     0.803.8
@angular-devkit/build-optimizer   0.803.8
@angular-devkit/build-webpack     0.803.8
@angular-devkit/core              7.2.4
@angular-devkit/schematics        7.2.4
@angular/cdk                      8.0.0
@angular/cli                      8.3.8
@angular/compiler-cli             9.0.0-next.9
@angular/material                 8.0.0
@angular/pwa                      0.12.4
@ngtools/webpack                  8.3.8
@schematics/angular               7.2.4
@schematics/update                0.803.8
rxjs                              6.5.3
typescript                        3.5.3
webpack                           4.39.2

Duplicate of #14980


Add special handling for display-name and namespaces in properties

Issue description / feature objectives
Currently set-parameters do not show properties or namespace in the markdown yaml header - but it is desirable to have the new "display-name" property visible and editable in the header.
In addition, when a new display name is added, an appropriate namespace should be assigned to it.
Furthermore, if any new property is added to a  control or part, a namespace should be assigned to it.
Caveats / Assumptions
We want to provide the needed editing capability without making the yaml header too busy or confusing.
No namespace values will ever be listed for properties in the yaml header - and instead they will either be as set in the source .json, or they will be specified by two new command line options described below.
Completion Criteria

Provide special handling of the display-name property in set-parameters by including it in the markdown yaml header along with name, values, etc.
The user may add or edit that display name
Provide new command line option display-name-ns for profile-assemble to specify the namespace for all display-name properties in the control.  If that value is provided on the command line, all display-name properties will have that namespace in the assembled profile.
Also provide new command line option default-namespace to use as the namespace for any added property that doesn't already have a namespace specified.  This only applies to profile-assemble.
If other -assemble commands should behave this way - please edit this list.

As discussed, during profile generate we will specify a CLI option default-ns. This would be shown in YAML header as a non-editable field. Currently we are assuming that "display-name" props of set-parameters will use this default ns. If in future they are using a different namespace then we will add another CLI option to specify the namespace for set-parameter properties (as we don't want to display the ns with each set-parameter property to keep set-parameter YAML headers simple).
For other properties at the control and subpart level, if the ns value is same as default-ns then the namespace for that property will not be shown in YAML header, else YAML header will show the specific ns for that property. If user creates or edits a property and don't specify a namespace in YAML header then the default-ns will be used during profile generate.
During profile-generate the default-ns CLI option will be optional. If not specified then the one from Makdown file will be used, else If specified then it will override the default namespace from markdown for all properties for which a namespace has not been explicitly specified in the YAML header. Properties with explicit namespace specified in YAML header will always have that namespace (not the default namespace).

To summarize the desired behavior:

A new command line option default-namespace will be added to profile generate and assemble
In order to avoid applying it to arbitrary properties in the NIST catalog, such as sort-id, for set-parameters it will only apply to the special property, display-name.  And for set-parameters, the only property ever shown is display-name - and its namespace is never shown since it is always the default
The default-namespace value is specified in the TRESTLE_GLOBAL section of the header, which is not directly editable by the user and is only for informational purposes.  It is never read by the code.
The properties in the ADD_PROPS section of the header are assumed to be special to the user and can have a default namespace implied
When a md file is first generated, if default-namespace is specified then only properties with a namespace different from the default will appear in the markdown
If a property ever appears in the markdown, it will forever be unchanged during generate/assemble unless changed by the user
If you generate again with a different default namespace, none of the listed namespace values will change.  In other words, default namespace only and always applies to add-props that have no ns value shown
If you assemble with no default namespace given, only props with ns in the markdown will have ns assigned
If you assemble with default namespace given, all add-props with no ns in markdown will get the default value


[FeatureREQ] Forwarding Mails without SMTP

Hi it's me again ;)
I'm not sure if I missed this information, but I couldn't find a solution for forwarding testmails without providing any smtp details. We often use sendmail on our server, but we wanna forward the testmails. Is there any way to do this without an external smtp relay?
Best regards
Maik

Hi :) No, this is not supported, you have to configure a relay SMTP server. Just to clarify, this also won't be supported in the future due to the technical requirements of a SMTP servers (it's not simple, and completely out of scope of what Mailpit is). Sorry.

Thanks for your reply.
sadly to hear, but understanding. I'll try to set a free account on service like mailgun, this will fix my issue.
stay tuned!
best regards
Maik


page setup missing

Hi, I've tried to install the server status, the config page are ikay, everything works fine but at the moment that I've finished setup, the page load like without the CSS, only texts

Hi, sorry for late reply. Can you post your config file - the WEB_URL in particular? It should look like this define("WEB_URL", "http://<IP_ADDRESS>/state"); //Used for links  - if it does not, try changing it.

Same issue here:
define("WEB_URL", "https%3A%2F%2Flaunchstatus.wracked.nl%2Fstatus"); //Used for links

The calls to urlencode() in install.php code should be removed for the two URL config options (url and policy_url). I see very few benefits from having these two parameters checked during installation. This would resolve #81, #91, #94 and #95.
@Pryx Do you want me to make up a quick PR?

Resolved by #101


Shantae: Half-Genie Hero [PCSE00950]

App summary

App name: Shantae: Half-Genie Hero
App serial: PCSE00950
App version: 1.05

Vita3K summary

Version: v0.1.6
Build number: 3101
Commit hash: https://github.com/vita3k/vita3k/commit/8623b51653f85f34766fe51499a9f622bc2cf26b
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Panderner
OS: Windows 11
CPU: Intel Core i3-12100F
GPU: NVIDIA GeForce GTX 1050 Ti
RAM: 16 GB

Issues

Major Graphical Issues
Game freezes at the title screen if DLCs are installed

Screenshots

Log
vita3k.log
Recommended labels

Ingame -
Graphics Bug

Screenshots from Pixel 6A

The textures of fire and The missiles don't come out, they just flash and it's almost impossible to see them and they don't come out square either.


BTHRM - Prevent mixed readings from both sensors

Prevent readings from internal sensor mixing into BT values
Mark events with src property
Show actual source of event in app

Great - thanks!


Error when evaluating ES module from Polyglot Source

Hello there, I am opening a new ticket regarding ECMAScript module support per the closing comment on Issue 92. I'm using GraalVM Java11 20.1.0 Darwin as my JRE. I first came across this resource, then came across Issue 92.
When I try to load an ES module like such from Java,
try (Context ctx = Context.create()) {
    File jsFile = new File(.../index.mjs);
    Source src = Source.newBuilder("js", jsFile)
            .name("index.mjs")
            .mimeType("application/javascript+module")
            .build();
    ctx.eval(src);
...

I get the following exception on ctx.eval(src): org.graalvm.polyglot.PolyglotException: Error: Operation is not allowed for: ...index.mjs
This is the case when I have a very simple index.mjs that looks exactly like that of @pete-by's comment from 02/19.
js/module1.mjs:
export const sqrt = Math.sqrt;
export function square(x) {
    return x * x;
}
export function diag(x, y) {
    return sqrt(square(x) + square(y));
}

js/index.mjs:
import { square, diag } from "module1";
console.log(square(11)); // 121
console.log(diag(4, 3)); // 5

I don't even hit the NoSuchFileException @pete-by encountered. I have already tried few permutations of having mimetype specified and between file type .mjs and .js when building the source and hit the same issue. I'm probably doing something wrong, but I'm not sure what. Any guidance would be greatly appreciated.
I have tried transpiling my index.mjs to VanillaJS via the webpack command in this article and am able to execute in the Polyglot context as VanillaJS, but it's proving to be a time-consuming step I'd like to avoid, hence the motivation of this post.

@fredho we have a few examples of ES module usages in our unit tests here. In fact, one of our unit tests loads a module that is very similar to your module1.mjs. In your example code above, you should create your polyglot Context with allowIo(true), which allows the JS engine to perform IO operations.

I see, I will give this a try. Thanks.


[Bug]: No notification on the Smartphone?

What happened and in what context?
I am not sure if it is a bug!
I have set the following:
[progress_notification]
percent: 20
The status is also updated, but I don't get any notification on my phone. I only see the update when I open the chat.
If needed, paste the relevant bot log contents here.
No response
If needed, paste the relevant bot journal contents here.
No response
What branch does this occur on?
master
Reports only on latest master or development branches!

[X] I have updated to latest development/master version before submitting the bug

@gitbreiti
a little workaround is commited in ad2d118
Could you please check it?

I have overwritten the 3 files. Is that correct? Now no image is sent.

@nlef
I have now overwritten all the files. Now a new notification comes: Status has been updated...
When restarting the bot I get the following message: Unknown/bad items in [telegram_ui] section: status_single_message:true
If I delete the section everything works fine! :)

After a reboot everything worked.
Closed.


docs: add markdown version of changelog

Changelog converted with
import re

with open("docs/source/changelog.rst") as f:
    text = f.read()

# :issue:`312` -> https://github.com/googleapis/python-bigquery-pandas/issues/312
c = re.compile(r":issue:`([0-9]+)`", flags=re.MULTILINE)

with open("docs/source/changelog.rst", "w", encoding="utf-8") as f:
    f.write(re.sub(c, r"`#\1 <https://github.com/googleapis/python-bigquery-pandas/issues/\1>`_", text))

and
pandoc --from=rst --to=gfm docs/source/changelog.rst -o CHANGELOG.md

Towards https://github.com/googleapis/python-bigquery-pandas/issues/353 and https://github.com/googleapis/python-bigquery-pandas/issues/367

Docs build locally:
(dev-3.9) ➜  python-bigquery-pandas git:(release-v0.16.0) nox -s docs
nox > Running session docs
nox > Creating virtual environment (virtualenv) using python3.8 in .nox/docs
nox > python -m pip install -r docs/requirements-docs.txt
nox > python -m pip install -e .
nox > sphinx-build -W -T -N -b html -d docs/source/_build/doctrees/ docs/source/ docs/source/_build/html/
Running Sphinx v4.1.2
making output directory... done
[autosummary] generating autosummary for: api.rst, changelog.md, contributing.rst, howto/authentication.rst, index.rst, install.rst, intro.rst, privacy.rst, reading.rst, writing.rst
loading intersphinx inventory from https://docs.python.org/objects.inv...
loading intersphinx inventory from https://pandas.pydata.org/pandas-docs/stable/objects.inv...
loading intersphinx inventory from https://pydata-google-auth.readthedocs.io/en/latest/objects.inv...
loading intersphinx inventory from https://google-auth.readthedocs.io/en/latest/objects.inv...
intersphinx inventory has moved: https://docs.python.org/objects.inv -> https://docs.python.org/3/objects.inv
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 10 source files that are out of date
updating environment: [new config] 10 added, 0 changed, 0 removed
reading sources... [ 10%] api
reading sources... [ 20%] changelog
/Users/swast/src/python-bigquery-pandas/.nox/docs/lib/python3.8/site-packages/recommonmark/parser.py:75: UserWarning: Container node skipped: type=document
  warn("Container node skipped: type={0}".format(mdnode.t))
reading sources... [ 30%] contributing
reading sources... [ 40%] howto/authentication
reading sources... [ 50%] index
reading sources... [ 60%] install
reading sources... [ 70%] intro
reading sources... [ 80%] privacy
reading sources... [ 90%] reading
reading sources... [100%] writing

looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [ 10%] api
writing output... [ 20%] changelog
writing output... [ 30%] contributing
writing output... [ 40%] howto/authentication
writing output... [ 50%] index
writing output... [ 60%] install
writing output... [ 70%] intro
writing output... [ 80%] privacy
writing output... [ 90%] reading
writing output... [100%] writing

generating indices... genindex done
writing additional pages... search done
copying static files... done
copying extra files... done
dumping search index in English (code: en)... done
dumping object inventory... done
build succeeded.

The HTML pages are in docs/source/_build/html.
nox > Session docs was successful.


Have anyone encounter this, could not comile dependency:p1_utils when I compile videoroom demo!

===> Fetching pc v1.14.0
===> Version cached at /home/lifan/.cache/rebar3/hex/hexpm/packages/pc-1.14.0.tar is up to date, reusing it
escript: exception error: undefined function erlang:get_stacktrace/0
in function  rebar3:main/1 (/tmp/cirrus-ci-build/src/rebar3.erl, line 72)
in call from escript:run/2 (escript.erl, line 750)
in call from escript:start/1 (escript.erl, line 277)
in call from init:start_em/1
in call from init:do_boot/3
** (Mix) Could not compile dependency :p1_utils, "/home/lifan/.mix/rebar3 bare compile --paths /home/lifan/workspace/elixir/membrane_demo/webrtc/videoroom/_build/dev/lib/*/ebin" command failed. Errors may have been logged above. You can recompile this dependency with "mix deps.compile p1_utils", update it with "mix deps.update p1_utils" or clean it with "mix deps.clean p1_utils"

Hi @lifan0913, what's your OS, Elixir and Erlang version?

OS is Debian 10 buster, elixir version is 1.13.0, erlang/OTP 24

No idea why this happens :( seems like an issue with https://github.com/processone/p1_utils/ . I'd try doing mix deps.update or try finding help there


Cannot run example in Android Studio 3.4.2

When I click the green triangle button to run "All in benchmark" in GUI, I'm getting
"/Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java" -ea -...
!!! JUnit version 3.8 or later expected:

java.lang.ClassNotFoundException: junit.framework.ComparisonFailure
	at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at com.intellij.rt.execution.junit.JUnitStarter.junitVersionChecks(JUnitStarter.java:221)
	at com.intellij.rt.execution.junit.JUnitStarter.canWorkWithJUnitVersion(JUnitStarter.java:207)
	at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:61)

Process finished with exit code 253

I tried to Run -> Edit Configurations' Android JUnit to JRE 1.8 (on my Mac) and also to API Level 28.

Ok I had to run tests on an emulator from actual test files.

Sometimes running from the package level will confuse Studio into trying to run on the host instead of the device, which results in that message.
You can try deleting that configuration by clicking the current test dropdown, clicking configurations, and deleting the one that's trying to run on the host, and trying again.


Android AccessToken from SignIn

Hi,
I can't get an accessToken on android from a signIn().
I got the following response object :

{
email
id
idToken
name
photo
scopes
serverAuthCode
}

but no accessToken.
On iOS I got :

{
accessToken
serverAuthCode
accessTokenExpirationDate
id
familyName
email
givenName
photo
name
idToken
}

Which seems normal according to the doc (not very convenient but w/e) BUT :
I tried to use the GoogleSignin.getAccessToken() function with my android device but the app crashes and says that the function doesn't exist.
 TypeError: _reactNativeGoogleSignin.GoogleSignin.getAccessToken is not a function
    at googleSignIn

Thank you !
Edit : I just saw @Elektro1776 opened a similar issue but closed it on his own (https://github.com/devfd/react-native-google-signin/issues/211)
Edit2 : Just saw the issue was fixed on https://github.com/devfd/react-native-google-signin/commit/25b197559509ae267168ba9013cc23e6e2598c7c but the changes are not there after installing the plugin

I have same issue. I have tried this commit so I can get accessToken from user object. But lock like some issue security here. Object user become nomal object and easy to access and change.
Before:
{
id: [getter/setter],
scope: [getter/setter],
...
}

Affer:
{
id: '123...',
scope: [
[getter/setter],
[getter/setter]
],
accessToken: 'ya29.GlsbBImkd...',
...
}

Same problem, ended up using something like this instead of hacking the library code:
import { Platform, NativeModules } from 'react-native'

const { RNGoogleSignin } = NativeModules;

...

GoogleSignin.signIn()
.then((user) => {

  if(Platform.OS.toLowerCase() === "ios"){

    // user.accessToken

  }else{

    RNGoogleSignin.getAccessToken(user).then((token) => {

        // token

    })
    .catch(err => {
      console.log('Could not get google access token: '+err);
    });

  }

})
.catch((err) => {
  console.log('Could not auth with google: ', err);
})
.done();

ya. same issue i was faced but solved by using this.
please take a look to link given below:
http://stackoverflow.com/questions/29765905/how-to-get-the-access-token-from-google-sign-in-javascript-sdk/43201100#43201100

Is this library not being actively maintained? https://github.com/devfd/react-native-google-signin/pull/174 fixes the issue referenced here. Its been several months since that PR was created.

@jordanfloyd I commented on the PR, it needs to resolve its conflicts before being merged.

@Kerumen what about https://github.com/devfd/react-native-google-signin/commit/25b197559509ae267168ba9013cc23e6e2598c7c then? This also fixes the issue referenced here. Timeline for an updated release? 0.9.0 is unusable as is because there is no way to retrieve the access token on android through the JS module. I have to pull in the native module myself.

@Kerumen awesome, thank you!

This issue is because Google Play Services is not up to date. Update it on the device and you should receive the token.

Yup this is fixed on the latest version. Please check out our example app.


[FEATURE] Permissions lists CLUSTER_PERMISSIONS/INDEX_PERMISSIONS should be retrieved at runtime from plugins

Is your feature request related to a problem?
Developers in the OpenSearch project have to update a file outside their repository to update the list of permissions that are shown in the security plugin's permission UX.
This makes testing updates difficult and requires coordination with an external team.  Since changes of this nature arrive toward the end of the release lifecycle it causes delays that can jeopardize the associated features.
What solution would you like?
It would be way easier if the list of permissions could be supplied at runtime by plugins and the security dashboard plugin could read, merge, sort, and present those values in the UX.
Do you have any additional context?
Reducing the amount of overhead to get common changes done in OpenSearch would benefit all plugins - additional plugins that aren't part of the OpenSearch distribution would have a mechanism to supply their own permissions to include in the list.
This discussion in a recent PR [1] required back and forth along with realigning expectations - and ultimately everything was already working as intended.

[1] https://github.com/opensearch-project/security-dashboards-plugin/pull/1182

@ylwu-amzn What do you think of this proposal?

Please prepare some RCF with high level design and discuss with all plugin teams and community.

Unfortunately I do not have bandwidth for this feature and I wanted to capture its essence before it slipped my mind.  This issue will be discussed during our open triage meetings [1] if you'd like to attend
[1] https://www.meetup.com/opensearch/events/bbrkzsydcpbkb/

@dagneyb This sounds like its related to registration points and other extensibility features - is there an overall effort this should be attached to?

Another time when this feature would have prevented a bad user experience

https://github.com/opensearch-project/security-dashboards-plugin/pull/1290

X-Posting from this comment thread: https://github.com/opensearch-project/security-dashboards-plugin/pull/1725#discussion_r1443038640
It is possible to get a full list of transport actions that have been registered in core (native actions + transport actions), but there are a couple of problems:

The actions aren't necessarily categorized the same way that the security plugin categorizes the actions. i.e. This new action is a cluster permission even though it begins with indices:. At some point, I think these need to be revisited to ensure that the action naming is consistent and works with the security plugin.
Not all transport actions are meant to be run by users, some may be internal and only invoked by a process. I think the security plugin's ConfigUpdateAction is one example of that.


Merge release/1.0.0 into develop

Merge release/1.0.0 into develop

May need to resolve the conflicts before merging in